-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/63470/
-----------------------------------------------------------
Review request for hive and Ashutosh Chauhan.
Bugs: HIVE-17767
https://issues.apache.org/jira/browse/HIVE-17767
Repository: hive-git
Description
-------
This patch does the following:
* Added back earlier patch to generate value generator
* Added logic to rewrite EXISTS/IN correlated subqueries into LEFT SEMI JOIN
* Remove SemiJoinTransposeRule (This rule pushes semi join underneath its left
join which might not be semantically correct thing to do)
Diffs
-----
itests/src/test/resources/testconfiguration.properties 462f332e99
ql/src/java/org/apache/hadoop/hive/ql/optimizer/calcite/HiveSubQRemoveRelBuilder.java
3a1897f4aa
ql/src/java/org/apache/hadoop/hive/ql/optimizer/calcite/rules/HiveRelDecorrelator.java
62125f0fb7
ql/src/java/org/apache/hadoop/hive/ql/optimizer/calcite/rules/HiveSubQueryRemoveRule.java
2dca6a25ac
ql/src/java/org/apache/hadoop/hive/ql/parse/CalcitePlanner.java 80351bef87
ql/src/test/queries/clientpositive/subquery_exists.q 19c42f0c29
ql/src/test/queries/clientpositive/subquery_in.q 4ba170a706
ql/src/test/results/clientpositive/constprog_partitioner.q.out 87618df902
ql/src/test/results/clientpositive/llap/dynamic_semijoin_reduction_2.q.out
87e08fbcde
ql/src/test/results/clientpositive/llap/explainuser_1.q.out 6e55acf0d8
ql/src/test/results/clientpositive/llap/lineage3.q.out 66cc6ad5a0
ql/src/test/results/clientpositive/llap/subquery_exists.q.out e206f0851e
ql/src/test/results/clientpositive/llap/subquery_in.q.out af42131bc2
ql/src/test/results/clientpositive/llap/subquery_multi.q.out 96fe17a05a
ql/src/test/results/clientpositive/llap/subquery_notin.q.out 8e2ca937af
ql/src/test/results/clientpositive/llap/subquery_scalar.q.out c89d053b4a
ql/src/test/results/clientpositive/llap/subquery_select.q.out 118f6ebccf
ql/src/test/results/clientpositive/llap/subquery_views.q.out a9a81133b5
ql/src/test/results/clientpositive/llap/vector_mapjoin_reduce.q.out
4e6f00f6b7
ql/src/test/results/clientpositive/masking_12.q.out 540c53e825
ql/src/test/results/clientpositive/masking_3.q.out 1114c80676
ql/src/test/results/clientpositive/masking_4.q.out 527da21610
ql/src/test/results/clientpositive/perf/spark/query10.q.out eb3a2f6699
ql/src/test/results/clientpositive/perf/spark/query16.q.out b74d721d41
ql/src/test/results/clientpositive/perf/spark/query35.q.out 8759b71b8c
ql/src/test/results/clientpositive/perf/spark/query69.q.out e4430beaac
ql/src/test/results/clientpositive/perf/spark/query94.q.out 43b8c77bdc
ql/src/test/results/clientpositive/perf/tez/query10.q.out cf3651b35b
ql/src/test/results/clientpositive/perf/tez/query14.q.out b2a45f155a
ql/src/test/results/clientpositive/perf/tez/query16.q.out a7b710d6e1
ql/src/test/results/clientpositive/perf/tez/query23.q.out 7112de61d9
ql/src/test/results/clientpositive/perf/tez/query35.q.out a72f57816e
ql/src/test/results/clientpositive/perf/tez/query69.q.out 591f3fcdb0
ql/src/test/results/clientpositive/perf/tez/query94.q.out 7674aa7f7c
ql/src/test/results/clientpositive/semijoin5.q.out 533c077f58
ql/src/test/results/clientpositive/spark/constprog_partitioner.q.out
b89f9f5905
ql/src/test/results/clientpositive/spark/spark_explainuser_1.q.out 76c74d9ab7
ql/src/test/results/clientpositive/spark/subquery_exists.q.out dafe5b6d5b
ql/src/test/results/clientpositive/spark/subquery_in.q.out 471c2ccd94
ql/src/test/results/clientpositive/spark/subquery_multi.q.out ff519fda09
ql/src/test/results/clientpositive/spark/subquery_notin.q.out 1b2c0880ae
ql/src/test/results/clientpositive/spark/subquery_scalar.q.out de005ada82
ql/src/test/results/clientpositive/spark/subquery_select.q.out 7d3a16b6ee
ql/src/test/results/clientpositive/spark/subquery_views.q.out 91e39913a7
ql/src/test/results/clientpositive/spark/vector_mapjoin_reduce.q.out
81af937e97
ql/src/test/results/clientpositive/subquery_exists.q.out c9f2a79041
ql/src/test/results/clientpositive/subquery_exists_having.q.out 2c41ff6c33
ql/src/test/results/clientpositive/subquery_in_having.q.out 6893442b61
ql/src/test/results/clientpositive/subquery_notexists.q.out 329573e8e1
ql/src/test/results/clientpositive/subquery_notexists_having.q.out 4d2b2fc873
ql/src/test/results/clientpositive/subquery_notin_having.q.out c321fe69ed
ql/src/test/results/clientpositive/subquery_unqualcolumnrefs.q.out 5c306f6b47
ql/src/test/results/clientpositive/vector_mapjoin_reduce.q.out ddea584990
Diff: https://reviews.apache.org/r/63470/diff/1/
Testing
-------
Thanks,
Vineet Garg
