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