Home | About | Sematext search-lucene.com search-hadoop.com
 Search Hadoop and all its subprojects:

Switch to Plain View
Hive, mail # dev - Review Request 13787: HIVE-5095: Hive needs new operator walker for parallelization/optimization for tez


Copy link to this message
-
Review Request 13787: HIVE-5095: Hive needs new operator walker for parallelization/optimization for tez
Vikram Dixit Kumaraswamy 2013-08-23, 23:45

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