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

Switch to Plain View
Hive, mail # dev - Re: Review Request: HIVE-2206: add a new optimizer for query correlation discovery and optimization


Copy link to this message
-
Re: Review Request: HIVE-2206: add a new optimizer for query correlation discovery and optimization
Yin Huai 2012-11-03, 01:50

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

(Updated Nov. 3, 2012, 1:50 a.m.)
Review request for hive.
Description
-------

This optimizer exploits intra-query correlations and merges multiple correlated MapReduce jobs into one jobs. Open a new request since I have been working on hive-git.
This addresses bug HIVE-2206.
    https://issues.apache.org/jira/browse/HIVE-2206
Diffs (updated)
-----

  common/src/java/org/apache/hadoop/hive/conf/HiveConf.java b8230e4
  conf/hive-default.xml.template 855f758
  ql/src/gen/thrift/gen-javabean/org/apache/hadoop/hive/ql/plan/api/OperatorType.java 8c9bd26
  ql/src/java/org/apache/hadoop/hive/ql/exec/BaseReduceSinkOperator.java PRE-CREATION
  ql/src/java/org/apache/hadoop/hive/ql/exec/CorrelationCompositeOperator.java PRE-CREATION
  ql/src/java/org/apache/hadoop/hive/ql/exec/CorrelationLocalSimulativeReduceSinkOperator.java PRE-CREATION
  ql/src/java/org/apache/hadoop/hive/ql/exec/CorrelationReducerDispatchOperator.java PRE-CREATION
  ql/src/java/org/apache/hadoop/hive/ql/exec/ExecReducer.java 18a9bd2
  ql/src/java/org/apache/hadoop/hive/ql/exec/GroupByOperator.java 46daeb2
  ql/src/java/org/apache/hadoop/hive/ql/exec/Operator.java 68302f8
  ql/src/java/org/apache/hadoop/hive/ql/exec/OperatorFactory.java 0c22141
  ql/src/java/org/apache/hadoop/hive/ql/exec/ReduceSinkOperator.java 919a140
  ql/src/java/org/apache/hadoop/hive/ql/exec/TableScanOperator.java 1469325
  ql/src/java/org/apache/hadoop/hive/ql/optimizer/CorrelationOptimizer.java PRE-CREATION
  ql/src/java/org/apache/hadoop/hive/ql/optimizer/CorrelationOptimizerUtils.java PRE-CREATION
  ql/src/java/org/apache/hadoop/hive/ql/optimizer/GenMapRedUtils.java edde378
  ql/src/java/org/apache/hadoop/hive/ql/optimizer/Optimizer.java d1555e2
  ql/src/java/org/apache/hadoop/hive/ql/parse/ParseContext.java 2bf284d
  ql/src/java/org/apache/hadoop/hive/ql/parse/SemanticAnalyzer.java 74b7d2b
  ql/src/java/org/apache/hadoop/hive/ql/plan/BaseReduceSinkDesc.java PRE-CREATION
  ql/src/java/org/apache/hadoop/hive/ql/plan/CorrelationCompositeDesc.java PRE-CREATION
  ql/src/java/org/apache/hadoop/hive/ql/plan/CorrelationLocalSimulativeReduceSinkDesc.java PRE-CREATION
  ql/src/java/org/apache/hadoop/hive/ql/plan/CorrelationReducerDispatchDesc.java PRE-CREATION
  ql/src/java/org/apache/hadoop/hive/ql/plan/MapredWork.java 5a9f064
  ql/src/java/org/apache/hadoop/hive/ql/plan/ReduceSinkDesc.java 16eb125
  ql/src/java/org/apache/hadoop/hive/ql/plan/TableScanDesc.java 9a95efd
  ql/src/test/org/apache/hadoop/hive/ql/exec/TestExecDriver.java 142f040
  ql/src/test/queries/clientpositive/correlationoptimizer1.q PRE-CREATION
  ql/src/test/queries/clientpositive/correlationoptimizer2.q PRE-CREATION
  ql/src/test/queries/clientpositive/correlationoptimizer3.q PRE-CREATION
  ql/src/test/queries/clientpositive/correlationoptimizer4.q PRE-CREATION
  ql/src/test/queries/clientpositive/correlationoptimizer5.q PRE-CREATION
  ql/src/test/results/clientpositive/correlationoptimizer1.q.out PRE-CREATION
  ql/src/test/results/clientpositive/correlationoptimizer2.q.out PRE-CREATION
  ql/src/test/results/clientpositive/correlationoptimizer3.q.out PRE-CREATION
  ql/src/test/results/clientpositive/correlationoptimizer4.q.out PRE-CREATION
  ql/src/test/results/clientpositive/correlationoptimizer5.q.out PRE-CREATION
  ql/src/test/results/compiler/plan/groupby1.q.xml e1eb209
  ql/src/test/results/compiler/plan/groupby2.q.xml 268eb90
  ql/src/test/results/compiler/plan/groupby3.q.xml 3e02d15
  ql/src/test/results/compiler/plan/groupby5.q.xml 89e1207

Diff: https://reviews.apache.org/r/7126/diff/
Testing
-------

All tests pass.
Thanks,

Yin Huai

+
Yin Huai 2012-11-12, 22:21
+
Yin Huai 2012-11-19, 19:51
+
Ashutosh Chauhan 2013-01-13, 01:00