----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43885/#review120363 -----------------------------------------------------------
ql/src/java/org/apache/hadoop/hive/ql/optimizer/calcite/HiveRexUtil.java (line 199) <https://reviews.apache.org/r/43885/#comment181800> Can you add a comment about why we are treating inverted and negated terms in same manner. My assumption was we can fold to false only in presence of negated terms, not inverted terms. ql/src/test/results/clientpositive/ppd_udf_col.q.out (line 117) <https://reviews.apache.org/r/43885/#comment181798> Seems like we missed opportunity to fold constant here. - Ashutosh Chauhan On Feb. 23, 2016, 5:27 p.m., Jesús Camacho Rodríguez wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/43885/ > ----------------------------------------------------------- > > (Updated Feb. 23, 2016, 5:27 p.m.) > > > Review request for hive and Ashutosh Chauhan. > > > Bugs: HIVE-13102 > https://issues.apache.org/jira/browse/HIVE-13102 > > > Repository: hive-git > > > Description > ------- > > HIVE-13102 > > > Diffs > ----- > > > ql/src/java/org/apache/hadoop/hive/ql/optimizer/calcite/HiveCalciteUtil.java > 58a7cff97cf8120b7afdb52af6a8f6cd07b408ea > ql/src/java/org/apache/hadoop/hive/ql/optimizer/calcite/HiveRexUtil.java > 3f6dd6a3738bdcb3c4756136965c54ea6f943f53 > > ql/src/java/org/apache/hadoop/hive/ql/optimizer/calcite/rules/HiveJoinPushTransitivePredicatesRule.java > 994af976f12f8b59c2379f23a33cf8fc94a2efc5 > > ql/src/java/org/apache/hadoop/hive/ql/optimizer/calcite/rules/HiveReduceExpressionsRule.java > 8f15ec7e5f075ceee1ca71b9a389837b9a31167e > > ql/src/java/org/apache/hadoop/hive/ql/optimizer/calcite/stats/HiveRelMdPredicates.java > 36d0b458d4c33707f8e3a5b823e35edf8c5d09d1 > ql/src/test/queries/clientpositive/constprog3.q PRE-CREATION > ql/src/test/queries/clientpositive/infer_join_preds.q PRE-CREATION > ql/src/test/results/clientpositive/constprog3.q.out PRE-CREATION > ql/src/test/results/clientpositive/fold_case.q.out > 90ea0afa49673910be4b28ce5f96e942b42936b0 > ql/src/test/results/clientpositive/infer_join_preds.q.out PRE-CREATION > ql/src/test/results/clientpositive/mergejoin.q.out > f2c04e2221c1e7164ffb7cfe49f2fe3e0536e321 > ql/src/test/results/clientpositive/partition_boolexpr.q.out > 5272f33621a45616895f81684e4f620f6b0ec765 > ql/src/test/results/clientpositive/ppd_udf_col.q.out > 2641f5c89a1a8839e8c3e5bfa5f4b6d756ec47fb > ql/src/test/results/clientpositive/tez/mergejoin.q.out > 14d04317d17684877566ae4aad29a862255ba02c > > Diff: https://reviews.apache.org/r/43885/diff/ > > > Testing > ------- > > > Thanks, > > Jesús Camacho Rodríguez > >