-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/1705/
-----------------------------------------------------------
(Updated 2011-11-04 19:48:37.718693)
Review request for hive, Yongqiang He, Ning Zhang, and Siying Dong.
Changes
-------
It's been a while, so I ran svn up and fixed the conflicts.
I ran the tests and verified they passed.
Summary
-------
I added checks before the type conversions for both joins and unions to either
throw an error or a warning depending if a bigint was going to be converted to
a double.
I also added a new variable hive.mapred.bigint.comparison.mode, which is set to
either strict or nonstrict to indicate whether an error or a warning should be
given when a bigint is converted to a double. This is instead of the original
implementation which used the variable hive.mapred.mode.
This addresses bug HIVE-2427.
https://issues.apache.org/jira/browse/HIVE-2427
Diffs (updated)
-----
trunk/common/src/java/org/apache/hadoop/hive/conf/HiveConf.java 1197375
trunk/conf/hive-default.xml 1197375
trunk/ql/src/java/org/apache/hadoop/hive/ql/parse/ErrorMsg.java 1197375
trunk/ql/src/java/org/apache/hadoop/hive/ql/parse/SemanticAnalyzer.java
1197375
trunk/ql/src/java/org/apache/hadoop/hive/ql/plan/ExprNodeGenericFuncDesc.java
1197375
trunk/ql/src/test/queries/clientnegative/compare_double_bigint.q 1197375
trunk/ql/src/test/queries/clientnegative/compare_double_bigint_join.q
PRE-CREATION
trunk/ql/src/test/queries/clientnegative/compare_double_bigint_union.q
PRE-CREATION
trunk/ql/src/test/queries/clientnegative/compare_string_bigint.q 1197375
trunk/ql/src/test/queries/clientnegative/compare_string_bigint_join.q
PRE-CREATION
trunk/ql/src/test/queries/clientnegative/compare_string_bigint_union.q
PRE-CREATION
trunk/ql/src/test/queries/clientpositive/filter_join_breaktask2.q 1197375
trunk/ql/src/test/results/clientnegative/compare_double_bigint.q.out 1197375
trunk/ql/src/test/results/clientnegative/compare_double_bigint_join.q.out
PRE-CREATION
trunk/ql/src/test/results/clientnegative/compare_double_bigint_union.q.out
PRE-CREATION
trunk/ql/src/test/results/clientnegative/compare_string_bigint.q.out 1197375
trunk/ql/src/test/results/clientnegative/compare_string_bigint_join.q.out
PRE-CREATION
trunk/ql/src/test/results/clientnegative/compare_string_bigint_union.q.out
PRE-CREATION
trunk/ql/src/test/results/clientpositive/filter_join_breaktask2.q.out 1197375
Diff: https://reviews.apache.org/r/1705/diff
Testing
-------
I added several more tests to demonstrate the errors.
I also modified a unit test in which a join was being done which involved a
comparison between a string and a bigint. I set
hive.mapred.bigint.comparison.mode to nonstrict, so the only change was the new
warning appeared in the output.
Thanks,
Kevin
