----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36353/ -----------------------------------------------------------
Review request for hive and Ashutosh Chauhan. Bugs: HIVE-10882 https://issues.apache.org/jira/browse/HIVE-10882 Repository: hive-git Description ------- CBO: Calcite Operator To Hive Operator (Calcite Return Path) empty filtersMap of join operator causes wrong results Diffs ----- ql/src/java/org/apache/hadoop/hive/ql/optimizer/calcite/HiveRelOptUtil.java 9ebb24ffc33c49b0d1479b17af99bba78ca4c5fc ql/src/java/org/apache/hadoop/hive/ql/optimizer/calcite/reloperators/HiveJoin.java 668960ec18a4ed8a40b4e08bc814f6ca14b13fd3 ql/src/java/org/apache/hadoop/hive/ql/optimizer/calcite/reloperators/HiveMultiJoin.java 911cedaccc1051a0b0dff56cee0c5758120fb139 ql/src/java/org/apache/hadoop/hive/ql/optimizer/calcite/reloperators/HiveSemiJoin.java dd1691c7482128bed15608b03df84eb20d4b06e6 ql/src/java/org/apache/hadoop/hive/ql/optimizer/calcite/rules/HiveJoinToMultiJoinRule.java c5e0e1153341218fa36bd3de2155d17478a17436 ql/src/java/org/apache/hadoop/hive/ql/optimizer/calcite/rules/HiveRelFieldTrimmer.java f72f67fec787ff8db729f998a96b1264bc6cf8cb ql/src/java/org/apache/hadoop/hive/ql/optimizer/calcite/translator/HiveOpConverter.java 84c6cc8b0e89dafc4cbbc9e6c337c3215d10ddbb ql/src/test/queries/clientpositive/fouter_join_ppr.q PRE-CREATION ql/src/test/results/clientpositive/cbo_rp_auto_join0.q.out 6fdc935af99971b89c58270014fab856c3e18d6e ql/src/test/results/clientpositive/cbo_rp_auto_join1.q.out 79b4650cc9605fd462b0f4c7080fdc860b32217e ql/src/test/results/clientpositive/cbo_rp_join0.q.out 1894110bca316d8b3fc44e3c3e087cd83ab01bf5 ql/src/test/results/clientpositive/cbo_rp_join1.q.out 69ce6d27571310b05b97ae159de50a5764b4c7f4 ql/src/test/results/clientpositive/fouter_join_ppr.q.out PRE-CREATION Diff: https://reviews.apache.org/r/36353/diff/ Testing ------- Existing tests + new test added with return path on. Thanks, Jesús Camacho Rodríguez