-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/1010/
-----------------------------------------------------------
(Updated 2011-07-19 03:15:17.006396)
Review request for hive and John Sichi.
Changes
-------
removed unnecessary imports from patch
Summary
-------
Grab the indexed tables during optimized query generation, grab the associated
path URIs, and keep those around in the Configuration object. When the job is
passed to ExecDriver, this data is extracted and used in HiveIndexedInputFormat
to decide whether to use the index file or delegate to the parent
(HiveInputFormat) class. Not sure if this is robust.
This addresses bug HIVE-2128.
https://issues.apache.org/jira/browse/HIVE-2128
Diffs (updated)
-----
ql/src/test/results/clientpositive/index_auto_self_join.q.out PRE-CREATION
ql/src/test/results/clientpositive/index_auto_mult_tables_compact.q.out
PRE-CREATION
ql/src/test/queries/clientpositive/index_auto_self_join.q PRE-CREATION
ql/src/test/results/clientpositive/index_auto_mult_tables.q.out PRE-CREATION
ql/src/java/org/apache/hadoop/hive/ql/plan/MapredWork.java a03a9a6
ql/src/test/queries/clientpositive/index_auto_mult_tables.q PRE-CREATION
ql/src/test/queries/clientpositive/index_auto_mult_tables_compact.q
PRE-CREATION
ql/src/java/org/apache/hadoop/hive/ql/index/HiveIndexResult.java b9b586e
ql/src/java/org/apache/hadoop/hive/ql/index/HiveIndexedInputFormat.java
f1ee95d
ql/src/java/org/apache/hadoop/hive/ql/index/bitmap/BitmapIndexHandler.java
61bbbf5
ql/src/java/org/apache/hadoop/hive/ql/index/compact/CompactIndexHandler.java
7c91946
ql/src/java/org/apache/hadoop/hive/ql/optimizer/physical/index/IndexWhereProcessor.java
dbc489f
ql/src/java/org/apache/hadoop/hive/ql/index/HiveIndexQueryContext.java
617723e
Diff: https://reviews.apache.org/r/1010/diff
Testing
-------
added new testcase index_auto_mult_tables.q
Thanks,
Syed
