-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/1194/
-----------------------------------------------------------
(Updated 2011-09-10 21:10:06.178279)
Review request for hive and John Sichi.
Changes
-------
Added order-by to queries for test determinism.
Summary
-------
This patch has defined a new AggregateIndexHandler which is used to optimize
the query plan for groupby queries.
This addresses bug HIVE-1694.
https://issues.apache.org/jira/browse/HIVE-1694
Diffs (updated)
-----
common/src/java/org/apache/hadoop/hive/conf/HiveConf.java 66ee0be
data/files/lineitem.txt PRE-CREATION
data/files/tbl.txt PRE-CREATION
ql/src/java/org/apache/hadoop/hive/ql/index/AggregateIndexHandler.java
PRE-CREATION
ql/src/java/org/apache/hadoop/hive/ql/index/HiveIndex.java 591c9ff
ql/src/java/org/apache/hadoop/hive/ql/index/bitmap/BitmapIndexHandler.java
5053576
ql/src/java/org/apache/hadoop/hive/ql/index/compact/CompactIndexHandler.java
7a00c00
ql/src/java/org/apache/hadoop/hive/ql/metadata/Hive.java bec8787
ql/src/java/org/apache/hadoop/hive/ql/optimizer/IndexUtils.java PRE-CREATION
ql/src/java/org/apache/hadoop/hive/ql/optimizer/Optimizer.java 590d69a
ql/src/java/org/apache/hadoop/hive/ql/optimizer/index/RewriteCanApplyCtx.java
PRE-CREATION
ql/src/java/org/apache/hadoop/hive/ql/optimizer/index/RewriteCanApplyProcFactory.java
PRE-CREATION
ql/src/java/org/apache/hadoop/hive/ql/optimizer/index/RewriteGBUsingIndex.java
PRE-CREATION
ql/src/java/org/apache/hadoop/hive/ql/optimizer/index/RewriteParseContextGenerator.java
PRE-CREATION
ql/src/java/org/apache/hadoop/hive/ql/optimizer/index/RewriteQueryUsingAggregateIndex.java
PRE-CREATION
ql/src/java/org/apache/hadoop/hive/ql/optimizer/index/RewriteQueryUsingAggregateIndexCtx.java
PRE-CREATION
ql/src/java/org/apache/hadoop/hive/ql/optimizer/physical/index/IndexWhereProcessor.java
dcdfb9e
ql/src/java/org/apache/hadoop/hive/ql/optimizer/physical/index/IndexWhereTaskDispatcher.java
699519b
ql/src/test/queries/clientpositive/ql_rewrite_gbtoidx.q PRE-CREATION
ql/src/test/results/clientpositive/ql_rewrite_gbtoidx.q.out PRE-CREATION
Diff: https://reviews.apache.org/r/1194/diff
Testing
-------
Thanks,
Prajakta
