You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@hive.apache.org by Ashutosh Chauhan <ha...@apache.org> on 2015/03/16 23:02:52 UTC

Review Request 32129: HIVE-9966 Get rid of customBucketMapJoin field from MapJoinDesc

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/32129/
-----------------------------------------------------------

Review request for hive and Vikram Dixit Kumaraswamy.


Bugs: HIVE-9966
    https://issues.apache.org/jira/browse/HIVE-9966


Repository: hive-git


Description
-------

Field & code around is redundant.


Diffs
-----

  ql/src/java/org/apache/hadoop/hive/ql/optimizer/ConvertJoinMapJoin.java ce43120 
  ql/src/java/org/apache/hadoop/hive/ql/optimizer/ReduceSinkMapJoinProc.java b184cf4 
  ql/src/java/org/apache/hadoop/hive/ql/optimizer/TezBucketJoinProcCtx.java 821f60c 
  ql/src/java/org/apache/hadoop/hive/ql/plan/CommonMergeJoinDesc.java f3728f1 
  ql/src/java/org/apache/hadoop/hive/ql/plan/MapJoinDesc.java 9fdd417 

Diff: https://reviews.apache.org/r/32129/diff/


Testing
-------

Refactor patch. Existing tests.


Thanks,

Ashutosh Chauhan