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



ql/src/java/org/apache/hadoop/hive/ql/optimizer/physical/index/IndexWhereProcessor.java
<https://reviews.apache.org/r/558/#comment1253>

    This does not work because the GenMRTableScan1.process() does not get 
called on these objects after we set these, as far as I can tell.



ql/src/java/org/apache/hadoop/hive/ql/optimizer/physical/index/IndexWhereProcessor.java
<https://reviews.apache.org/r/558/#comment1254>

    At this point, the inputFormat for this work is actually null.  I do not 
know which work I need to change the inputFormat for, and I can't think of an 
easy way to get to other work.


- Russell


On 2011-04-29 00:01:06, Russell Melick wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/558/
> -----------------------------------------------------------
> 
> (Updated 2011-04-29 00:01:06)
> 
> 
> Review request for hive.
> 
> 
> Summary
> -------
> 
> Review request for HIVE-1644.12.patch
> 
> 
> This addresses bug HIVE-1644.
>     https://issues.apache.org/jira/browse/HIVE-1644
> 
> 
> Diffs
> -----
> 
>   common/src/java/org/apache/hadoop/hive/conf/HiveConf.java f77f46c 
>   conf/hive-default.xml 6bd615e 
>   eclipse-templates/.classpath 8d2dc52 
>   ql/src/java/org/apache/hadoop/hive/ql/Driver.java ca337a8 
>   ql/src/java/org/apache/hadoop/hive/ql/exec/ExecDriver.java 24e16e4 
>   ql/src/java/org/apache/hadoop/hive/ql/exec/TableScanOperator.java c02d90b 
>   ql/src/java/org/apache/hadoop/hive/ql/index/AbstractIndexHandler.java 
> dd0186d 
>   ql/src/java/org/apache/hadoop/hive/ql/index/HiveIndexHandler.java 411b78f 
>   ql/src/java/org/apache/hadoop/hive/ql/index/HiveIndexQueryContext.java 
> PRE-CREATION 
>   
> ql/src/java/org/apache/hadoop/hive/ql/index/compact/CompactIndexHandler.java 
> f90d64f 
>   ql/src/java/org/apache/hadoop/hive/ql/metadata/Table.java 50db44c 
>   ql/src/java/org/apache/hadoop/hive/ql/optimizer/GenMRTableScan1.java 
> 6162676 
>   
> ql/src/java/org/apache/hadoop/hive/ql/optimizer/physical/IndexWhereResolver.java
>  PRE-CREATION 
>   
> ql/src/java/org/apache/hadoop/hive/ql/optimizer/physical/PhysicalOptimizer.java
>  0ae9fa2 
>   
> ql/src/java/org/apache/hadoop/hive/ql/optimizer/physical/index/IndexWhereProcCtx.java
>  PRE-CREATION 
>   
> ql/src/java/org/apache/hadoop/hive/ql/optimizer/physical/index/IndexWhereProcessor.java
>  PRE-CREATION 
>   
> ql/src/java/org/apache/hadoop/hive/ql/optimizer/physical/index/IndexWhereTaskDispatcher.java
>  PRE-CREATION 
>   ql/src/java/org/apache/hadoop/hive/ql/parse/ParseContext.java 374e123 
>   ql/src/java/org/apache/hadoop/hive/ql/parse/SemanticAnalyzer.java 2207ac4 
>   ql/src/java/org/apache/hadoop/hive/ql/plan/MapredWork.java 73391e9 
>   ql/src/test/queries/clientpositive/index_auto.q PRE-CREATION 
>   ql/src/test/queries/clientpositive/index_auto_file_format.q PRE-CREATION 
>   ql/src/test/queries/clientpositive/index_auto_multiple.q PRE-CREATION 
>   ql/src/test/queries/clientpositive/index_auto_partitioned.q PRE-CREATION 
>   ql/src/test/queries/clientpositive/index_auto_test_if_used.q PRE-CREATION 
>   ql/src/test/queries/clientpositive/index_auto_unused.q PRE-CREATION 
>   ql/src/test/results/clientpositive/index_auto.q.out PRE-CREATION 
>   ql/src/test/results/clientpositive/index_auto_file_format.q.out 
> PRE-CREATION 
>   ql/src/test/results/clientpositive/index_auto_multiple.q.out PRE-CREATION 
>   ql/src/test/results/clientpositive/index_auto_partitioned.q.out 
> PRE-CREATION 
>   ql/src/test/results/clientpositive/index_auto_test_if_used.q.out 
> PRE-CREATION 
>   ql/src/test/results/clientpositive/index_auto_unused.q.out PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/558/diff
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Russell
> 
>

Reply via email to