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

Reply via email to