-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/72526/
-----------------------------------------------------------
(Updated June 8, 2020, 12:52 p.m.)
Review request for hive and Jesús Camacho Rodríguez.
Bugs: HIVE-23493
https://issues.apache.org/jira/browse/HIVE-23493
Repository: hive-git
Description
-------
Rewrite plan to join back tables with many projected columns joined multiple
times
Diffs (updated)
-----
common/src/java/org/apache/hadoop/hive/conf/HiveConf.java 1464d6aa7a
ql/src/java/org/apache/hadoop/hive/ql/optimizer/calcite/rules/HiveCardinalityPreservingJoinOptimization.java
PRE-CREATION
ql/src/java/org/apache/hadoop/hive/ql/optimizer/calcite/rules/HiveCardinalityPreservingJoinRule.java
PRE-CREATION
ql/src/java/org/apache/hadoop/hive/ql/optimizer/calcite/rules/HiveFieldTrimmerRule.java
73ff1bccf2
ql/src/java/org/apache/hadoop/hive/ql/optimizer/calcite/rules/HiveRelFieldTrimmer.java
19ce3ea223
ql/src/java/org/apache/hadoop/hive/ql/parse/CalcitePlanner.java 377e8280e5
ql/src/test/results/clientpositive/perf/tez/constraints/cbo_query11.q.out
0136ee4bb5
ql/src/test/results/clientpositive/perf/tez/constraints/cbo_query4.q.out
987a0f348e
ql/src/test/results/clientpositive/perf/tez/constraints/cbo_query74.q.out
289e5d2569
ql/src/test/results/clientpositive/perf/tez/constraints/query11.q.out
7f9df5e8af
ql/src/test/results/clientpositive/perf/tez/constraints/query4.q.out
585f4d6b9c
ql/src/test/results/clientpositive/perf/tez/constraints/query74.q.out
39c76fc82c
Diff: https://reviews.apache.org/r/72526/diff/5/
Changes: https://reviews.apache.org/r/72526/diff/4-5/
Testing (updated)
-------
mvn test -Dtest.output.overwrite -DskipSparkTests
-Dtest=TestMiniLlapLocalCliDriver -Dqfile=cardinality_preserving_join_opt.q -pl
itests/qtest -Pitests
mvn test -Dtest.output.overwrite -DskipSparkTests
-Dtest=TestTezPerfConstraintsCliDriver
-Dqfile=cbo_query4.q,cbo_query11.q,cbo_query74.q,query4.q,query11.q,query74.q
-pl itests/qtest -Pitests
Thanks,
Krisztian Kasa
