----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46974/#review131664 -----------------------------------------------------------
ql/src/java/org/apache/hadoop/hive/ql/optimizer/calcite/rules/HiveReduceExpressionsRule.java (line 445) <https://reviews.apache.org/r/46974/#comment195703> Does this mean it will not handle multi AND/OR, between, in etc? ql/src/java/org/apache/hadoop/hive/ql/optimizer/calcite/rules/HiveReduceExpressionsRule.java (line 517) <https://reviews.apache.org/r/46974/#comment195702> In such cases shall we fold these predicate and add: always false? In machine generated queries these patterns might be encountered. ql/src/java/org/apache/hadoop/hive/ql/optimizer/calcite/rules/HiveUnionPullUpConstantsRule.java (line 107) <https://reviews.apache.org/r/46974/#comment195701> This will also update type of expression if needed, correct? e.g. pushing 86L to column of type int. I think thats probably ok. Can you add a comment for that. - Ashutosh Chauhan On May 4, 2016, 1:08 p.m., Jesús Camacho Rodríguez wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/46974/ > ----------------------------------------------------------- > > (Updated May 4, 2016, 1:08 p.m.) > > > Review request for hive and Ashutosh Chauhan. > > > Bugs: HIVE-13639 > https://issues.apache.org/jira/browse/HIVE-13639 > > > Repository: hive-git > > > Description > ------- > > HIVE-13639 > > > Diffs > ----- > > > ql/src/java/org/apache/hadoop/hive/ql/optimizer/calcite/rules/HiveReduceExpressionsRule.java > 9006f45784359ff22075bac61f4d92a51309301c > > ql/src/java/org/apache/hadoop/hive/ql/optimizer/calcite/rules/HiveUnionPullUpConstantsRule.java > PRE-CREATION > ql/src/java/org/apache/hadoop/hive/ql/parse/CalcitePlanner.java > 8e00e0b140d7d40f25299e10be26c6eaf0231745 > ql/src/test/queries/clientpositive/cbo_union_view.q PRE-CREATION > ql/src/test/results/clientpositive/cbo_union_view.q.out PRE-CREATION > ql/src/test/results/clientpositive/union_view.q.out > badd2091e8cd74ea06dd6228603755d4a5098734 > > Diff: https://reviews.apache.org/r/46974/diff/ > > > Testing > ------- > > > Thanks, > > Jesús Camacho Rodríguez > >
