-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/1759/
-----------------------------------------------------------
(Updated 2011-09-12 17:27:12.064230)
Review request for hive and John Sichi.
Summary
-------
while executing UDF if the implementation jar is not present in MR class path
it is throwing nullpointer exception instead of throwing nullpointer exception
throwing meaning full exception with the required details.
This addresses bug HIVE-2182.
https://issues.apache.org/jira/browse/HIVE-2182
Diffs (updated)
-----
trunk/ql/src/java/org/apache/hadoop/hive/ql/udf/generic/GenericUDFBridge.java
1169710
trunk/ql/src/test/queries/clientnegative/udfnull.q PRE-CREATION
trunk/ql/src/test/results/clientnegative/udfnull.q.out PRE-CREATION
trunk/ql/src/test/results/compiler/plan/cast1.q.xml 1169710
trunk/ql/src/test/results/compiler/plan/groupby1.q.xml 1169710
trunk/ql/src/test/results/compiler/plan/groupby2.q.xml 1169710
trunk/ql/src/test/results/compiler/plan/groupby3.q.xml 1169710
trunk/ql/src/test/results/compiler/plan/groupby4.q.xml 1169710
trunk/ql/src/test/results/compiler/plan/groupby5.q.xml 1169710
trunk/ql/src/test/results/compiler/plan/groupby6.q.xml 1169710
trunk/ql/src/test/results/compiler/plan/input20.q.xml 1169710
trunk/ql/src/test/results/compiler/plan/input8.q.xml 1169710
trunk/ql/src/test/results/compiler/plan/join2.q.xml 1169710
trunk/ql/src/test/results/compiler/plan/sample1.q.xml 1169710
trunk/ql/src/test/results/compiler/plan/sample2.q.xml 1169710
trunk/ql/src/test/results/compiler/plan/sample3.q.xml 1169710
trunk/ql/src/test/results/compiler/plan/sample4.q.xml 1169710
trunk/ql/src/test/results/compiler/plan/sample5.q.xml 1169710
trunk/ql/src/test/results/compiler/plan/sample6.q.xml 1169710
trunk/ql/src/test/results/compiler/plan/sample7.q.xml 1169710
trunk/ql/src/test/results/compiler/plan/udf1.q.xml 1169710
trunk/ql/src/test/results/compiler/plan/udf4.q.xml 1169710
trunk/ql/src/test/results/compiler/plan/udf6.q.xml 1169710
Diff: https://reviews.apache.org/r/1759/diff
Testing
-------
Added unit tests
Thanks,
chinna
