----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58181/#review171246 -----------------------------------------------------------
ql/src/java/org/apache/hadoop/hive/ql/parse/IdentifiersParser.g Line 590 (original), 597 (patched) <https://reviews.apache.org/r/58181/#comment244142> Can you also confirm the precedence of is distinct with other operators? Now, it is the same precedence as equal. try 1 is not distinct from 2 is not distinct from false, etc. ql/src/test/queries/clientpositive/is_distinct_from.q Lines 1 (patched) <https://reviews.apache.org/r/58181/#comment244143> Can we compare a column with a constant in is distinct from? or two columns? Or it should be limited to compare constants only? ql/src/test/results/clientpositive/udf_equal.q.out Lines 46 (patched) <https://reviews.apache.org/r/58181/#comment244144> OK, if it is a synonyms of <=>, then the precedence is correct. - pengcheng xiong On April 6, 2017, 8:02 p.m., Vineet Garg wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/58181/ > ----------------------------------------------------------- > > (Updated April 6, 2017, 8:02 p.m.) > > > Review request for hive, Ashutosh Chauhan, Jesús Camacho Rodríguez, and > Pengcheng Xu. > > > Bugs: HIVE-15986 > https://issues.apache.org/jira/browse/HIVE-15986 > > > Repository: hive-git > > > Description > ------- > > This patch adds support for 'is distinct from' and 'is not distinct from'. > > > Diffs > ----- > > itests/src/test/resources/testconfiguration.properties 7a70c9c > ql/src/java/org/apache/hadoop/hive/ql/exec/FunctionRegistry.java ccfb455 > > ql/src/java/org/apache/hadoop/hive/ql/optimizer/calcite/translator/SqlFunctionConverter.java > 85450c9 > ql/src/java/org/apache/hadoop/hive/ql/parse/HiveParser.g d98a663 > ql/src/java/org/apache/hadoop/hive/ql/parse/IdentifiersParser.g 8c4ee8a > ql/src/java/org/apache/hadoop/hive/ql/parse/TypeCheckProcFactory.java > f979c14 > ql/src/test/queries/clientpositive/is_distinct_from.q PRE-CREATION > ql/src/test/results/clientpositive/llap/is_distinct_from.q.out PRE-CREATION > ql/src/test/results/clientpositive/show_functions.q.out 68e248a > ql/src/test/results/clientpositive/udf_equal.q.out 52bd843 > > > Diff: https://reviews.apache.org/r/58181/diff/4/ > > > Testing > ------- > > Added new tests > Pre-commit testing > > > Thanks, > > Vineet Garg > >
