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

Posted to dev@hive.apache.org by Vikram Dixit Kumaraswamy <vi...@gmail.com> on 2013/08/24 01:45:23 UTC

Review Request 13787: HIVE-5095: Hive needs new operator walker for parallelization/optimization for tez

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

Review request for hive and Gunther Hagleitner.


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


Repository: hive-git


Description
-------

For tez to compute the number of reducers, we should be walking the operator tree in a topological fashion so that the reducers down the tree get the estimate from all parents. However, the current walkers in hive only walk the operator tree in a depth-first fashion. We need to add a new walker for the topological walk. Also, since information about the parent operators needs to be propagated on a per parent basis, we need to retain some context across operators to be passed to the child which the walker will co-ordinate.


Diffs
-----

  common/src/java/org/apache/hadoop/hive/conf/HiveConf.java 7408a5a 
  ql/src/java/org/apache/hadoop/hive/ql/exec/Operator.java ca48f5e 
  ql/src/java/org/apache/hadoop/hive/ql/exec/ReduceSinkOperator.java 6a538e8 
  ql/src/java/org/apache/hadoop/hive/ql/exec/TableScanOperator.java 6ee13ec 
  ql/src/java/org/apache/hadoop/hive/ql/exec/Utilities.java f3c34d1 
  ql/src/java/org/apache/hadoop/hive/ql/exec/mr/MapRedTask.java 49a0ee3 
  ql/src/java/org/apache/hadoop/hive/ql/optimizer/GenMapRedUtils.java 7433ddc 
  ql/src/java/org/apache/hadoop/hive/ql/optimizer/OpProcessor.java PRE-CREATION 
  ql/src/java/org/apache/hadoop/hive/ql/optimizer/physical/OpProcContext.java PRE-CREATION 
  ql/src/java/org/apache/hadoop/hive/ql/parse/GenOpGraphWalker.java PRE-CREATION 
  ql/src/java/org/apache/hadoop/hive/ql/parse/GenTezProcContext.java 827637a 
  ql/src/java/org/apache/hadoop/hive/ql/parse/GenTezWork.java ff8b17b 
  ql/src/java/org/apache/hadoop/hive/ql/parse/MapReduceCompiler.java c1c1da5 
  ql/src/java/org/apache/hadoop/hive/ql/parse/TaskCompiler.java 248eb03 
  ql/src/java/org/apache/hadoop/hive/ql/parse/TezCompiler.java 5abedfe 
  ql/src/java/org/apache/hadoop/hive/ql/plan/PlanUtils.java 5fd8d828 

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


Testing
-------


Thanks,

Vikram Dixit Kumaraswamy