----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55148/#review160893 -----------------------------------------------------------
ql/src/java/org/apache/hadoop/hive/ql/optimizer/calcite/HiveSubQRemoveRelBuilder.java (line 151) <https://reviews.apache.org/r/55148/#comment232131> Why do we have this HiveSubQRemoveRelBuilder? If there was a bug in the original RelBuilder, would not be sufficient to override that method in the HiveRelBuilder? Maybe this is not part of this issue and it is a follow-up; I am just trying to keep things tidy. ql/src/java/org/apache/hadoop/hive/ql/optimizer/calcite/rules/HiveSubQueryRemoveRule.java (line 78) <https://reviews.apache.org/r/55148/#comment232132> Why do we have this HiveSubQRemoveRelBuilder? If there was a bug in the original RelBuilder, would not be sufficient to override that method in the HiveRelBuilder? That would avoid bringing the whole class. Maybe this is not part of this issue and it is a follow-up; I am just trying to keep things tidy. ql/src/java/org/apache/hadoop/hive/ql/parse/CalcitePlanner.java (line 2084) <https://reviews.apache.org/r/55148/#comment232133> Comment should be removed? - Jesús Camacho Rodríguez On Jan. 6, 2017, 11:54 p.m., Vineet Garg wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/55148/ > ----------------------------------------------------------- > > (Updated Jan. 6, 2017, 11:54 p.m.) > > > Review request for hive, Ashutosh Chauhan and Jesús Camacho Rodríguez. > > > Bugs: HIVE-15481 > https://issues.apache.org/jira/browse/HIVE-15481 > > > Repository: hive-git > > > Description > ------- > > This patch lifts the following restrictions: > Restriction.8.m We allow only 1 SubQuery expression per Query. > Restriction.9.m We will not support nested SubQuery expressions. > Check.4.h For Exists and Not Exists, the Sub Query must have 1 or more > correlated predicates > Check.12.h SubQuery predicates cannot only refer to Parent Query columns. > > > Diffs > ----- > > data/files/part_tiny_nulls.txt PRE-CREATION > itests/src/test/resources/testconfiguration.properties 70e7197 > > ql/src/java/org/apache/hadoop/hive/ql/optimizer/calcite/HiveSubQRemoveRelBuilder.java > e028a99 > > ql/src/java/org/apache/hadoop/hive/ql/optimizer/calcite/reloperators/HiveFilter.java > ce207da > > ql/src/java/org/apache/hadoop/hive/ql/optimizer/calcite/rules/HiveSubQueryRemoveRule.java > 5f58aae > ql/src/java/org/apache/hadoop/hive/ql/parse/CalcitePlanner.java fdb468d > ql/src/java/org/apache/hadoop/hive/ql/parse/QBSubQuery.java cfac6c0 > ql/src/test/queries/clientnegative/subquery_restrictions.q 80870d7 > ql/src/test/queries/clientnegative/subquery_subquery_chain_exists.q > PRE-CREATION > ql/src/test/queries/clientpositive/perf/query16.q PRE-CREATION > ql/src/test/queries/clientpositive/perf/query23.q PRE-CREATION > ql/src/test/queries/clientpositive/perf/query33.q PRE-CREATION > ql/src/test/queries/clientpositive/perf/query45.q 0b34bfd > ql/src/test/queries/clientpositive/perf/query56.q PRE-CREATION > ql/src/test/queries/clientpositive/perf/query60.q PRE-CREATION > ql/src/test/queries/clientpositive/perf/query69.q PRE-CREATION > ql/src/test/queries/clientpositive/perf/query83.q PRE-CREATION > ql/src/test/queries/clientpositive/subquery_exists.q be80721 > ql/src/test/queries/clientpositive/subquery_in.q 5b22dce > ql/src/test/queries/clientpositive/subquery_in_having.q 2433556 > ql/src/test/queries/clientpositive/subquery_multi.q PRE-CREATION > ql/src/test/queries/clientpositive/subquery_notexists.q 2d7e9ce > ql/src/test/queries/clientpositive/subquery_notin.q c29e63f > ql/src/test/results/clientnegative/subquery_corr_grandparent.q.out fb72270 > ql/src/test/results/clientnegative/subquery_restrictions.q.out a546d49 > ql/src/test/results/clientnegative/subquery_subquery_chain_exists.q.out > PRE-CREATION > ql/src/test/results/clientpositive/llap/explainuser_1.q.out fa54bb7 > ql/src/test/results/clientpositive/llap/lineage3.q.out 72a9344 > ql/src/test/results/clientpositive/llap/subquery_exists.q.out b132cb6 > ql/src/test/results/clientpositive/llap/subquery_in.q.out e71add5 > ql/src/test/results/clientpositive/llap/subquery_multi.q.out PRE-CREATION > ql/src/test/results/clientpositive/llap/subquery_notin.q.out 252b058 > ql/src/test/results/clientpositive/llap/vector_groupby_mapjoin.q.out > 27f32db > ql/src/test/results/clientpositive/perf/query16.q.out PRE-CREATION > ql/src/test/results/clientpositive/perf/query23.q.out PRE-CREATION > ql/src/test/results/clientpositive/perf/query33.q.out PRE-CREATION > ql/src/test/results/clientpositive/perf/query56.q.out PRE-CREATION > ql/src/test/results/clientpositive/perf/query60.q.out PRE-CREATION > ql/src/test/results/clientpositive/perf/query69.q.out PRE-CREATION > ql/src/test/results/clientpositive/perf/query83.q.out PRE-CREATION > ql/src/test/results/clientpositive/spark/subquery_exists.q.out c28a218 > ql/src/test/results/clientpositive/spark/subquery_in.q.out 36e3e6e > ql/src/test/results/clientpositive/subquery_exists.q.out 1019e7a > ql/src/test/results/clientpositive/subquery_in_having.q.out 1ab7e7f > ql/src/test/results/clientpositive/subquery_notexists.q.out 6ec3b46 > ql/src/test/results/clientpositive/subquery_notin_having.q.out 9f72cc9 > ql/src/test/results/clientpositive/vector_groupby_mapjoin.q.out af42e41 > > Diff: https://reviews.apache.org/r/55148/diff/ > > > Testing > ------- > > *Added new tests > *Verfied result correctness with postgress (manually) > *Pre-commit testing > > > Thanks, > > Vineet Garg > >
