----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/1515/#review1510 -----------------------------------------------------------
It's OK if you think it's equivalent. Just make sure you print to console's error output stream, instead of Log.WARN - Siying On 2011-08-17 18:34:44, Kevin Wilfong wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/1515/ > ----------------------------------------------------------- > > (Updated 2011-08-17 18:34:44) > > > Review request for hive and Siying Dong. > > > Summary > ------- > > I added a check in the code for equality expressions (includes inequalities) > with operands of different types, that throws an error or logs a warning, > depending on strict mode, if one operand is a string or double and the other > is a bigint. > > > This addresses bug HIVE-2378. > https://issues.apache.org/jira/browse/HIVE-2378 > > > Diffs > ----- > > trunk/ql/src/java/org/apache/hadoop/hive/ql/parse/ErrorMsg.java 1158835 > > trunk/ql/src/java/org/apache/hadoop/hive/ql/udf/generic/GenericUDFBaseCompare.java > 1158835 > trunk/ql/src/test/queries/clientnegative/compare_double_bigint.q > PRE-CREATION > trunk/ql/src/test/queries/clientnegative/compare_string_bigint.q > PRE-CREATION > trunk/ql/src/test/results/clientnegative/compare_double_bigint.q.out > PRE-CREATION > trunk/ql/src/test/results/clientnegative/compare_string_bigint.q.out > PRE-CREATION > > Diff: https://reviews.apache.org/r/1515/diff > > > Testing > ------- > > I added two tests (one for strings and one for doubles) to record the issue. > > I also verified the unit tests still run. > > > Thanks, > > Kevin > >
