----------------------------------------------------------- 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