Re: Review Request 30151: Remove Extract Operator & its friends from codebase.

[Navis Ryu <na...@nexr.com>]

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

Ship it!


Ship It!

- Navis Ryu


On Jan. 30, 2015, 7:46 p.m., Ashutosh Chauhan wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/30151/
> -----------------------------------------------------------
> 
> (Updated Jan. 30, 2015, 7:46 p.m.)
> 
> 
> Review request for hive and Navis Ryu.
> 
> 
> Bugs: HIVE-9416
>     https://issues.apache.org/jira/browse/HIVE-9416
> 
> 
> Repository: hive-git
> 
> 
> Description
> -------
> 
> Remove Extract Operator & its friends from codebase.
> 
> 
> Diffs
> -----
> 
>   ql/src/java/org/apache/hadoop/hive/ql/exec/ExtractOperator.java c299d3a 
>   ql/src/java/org/apache/hadoop/hive/ql/exec/OperatorFactory.java f3c382a 
>   ql/src/java/org/apache/hadoop/hive/ql/exec/PTFOperator.java 2e6a880 
>   ql/src/java/org/apache/hadoop/hive/ql/exec/vector/VectorExtractOperator.java 7f4bb64 
>   ql/src/java/org/apache/hadoop/hive/ql/optimizer/BucketingSortingReduceSinkOptimizer.java 24ca89f 
>   ql/src/java/org/apache/hadoop/hive/ql/optimizer/ConstantPropagateProcFactory.java f36f843 
>   ql/src/java/org/apache/hadoop/hive/ql/optimizer/SortedDynPartitionOptimizer.java 137956c 
>   ql/src/java/org/apache/hadoop/hive/ql/optimizer/correlation/CorrelationUtilities.java 630a9eb 
>   ql/src/java/org/apache/hadoop/hive/ql/optimizer/correlation/ReduceSinkDeDuplication.java 3fead79 
>   ql/src/java/org/apache/hadoop/hive/ql/optimizer/lineage/OpProcFactory.java adca50d 
>   ql/src/java/org/apache/hadoop/hive/ql/optimizer/physical/BucketingSortingInferenceOptimizer.java 7954767 
>   ql/src/java/org/apache/hadoop/hive/ql/optimizer/physical/BucketingSortingOpProcFactory.java cf02bec 
>   ql/src/java/org/apache/hadoop/hive/ql/optimizer/physical/Vectorizer.java 94b4621 
>   ql/src/java/org/apache/hadoop/hive/ql/parse/SemanticAnalyzer.java c9a5ce5 
>   ql/src/java/org/apache/hadoop/hive/ql/plan/ExtractDesc.java 6762155 
>   ql/src/java/org/apache/hadoop/hive/ql/plan/SelectDesc.java fa6b548 
>   ql/src/test/org/apache/hadoop/hive/ql/exec/TestExecDriver.java 41862e6 
>   ql/src/test/results/clientpositive/bucket1.q.out 13ec735 
>   ql/src/test/results/clientpositive/bucket2.q.out 32a77c3 
>   ql/src/test/results/clientpositive/bucket3.q.out ff7173e 
>   ql/src/test/results/clientpositive/bucket4.q.out b99d12f 
>   ql/src/test/results/clientpositive/bucket5.q.out 5992d6d 
>   ql/src/test/results/clientpositive/bucket6.q.out 5b23d7d 
>   ql/src/test/results/clientpositive/bucketsortoptimize_insert_1.q.out 75de953 
>   ql/src/test/results/clientpositive/bucketsortoptimize_insert_2.q.out 599b8b9 
>   ql/src/test/results/clientpositive/bucketsortoptimize_insert_3.q.out 7456ab0 
>   ql/src/test/results/clientpositive/bucketsortoptimize_insert_4.q.out fd99597 
>   ql/src/test/results/clientpositive/bucketsortoptimize_insert_5.q.out 8130ab9 
>   ql/src/test/results/clientpositive/bucketsortoptimize_insert_6.q.out 627aba0 
>   ql/src/test/results/clientpositive/disable_merge_for_bucketing.q.out 9b058c8 
>   ql/src/test/results/clientpositive/dynpart_sort_opt_vectorization.q.out 0baa446 
>   ql/src/test/results/clientpositive/dynpart_sort_optimization.q.out 494bfa3 
>   ql/src/test/results/clientpositive/encrypted/encryption_insert_partition_dynamic.q.out b6e7b88 
>   ql/src/test/results/clientpositive/encrypted/encryption_insert_partition_static.q.out fc6d2ae 
>   ql/src/test/results/clientpositive/load_dyn_part2.q.out 26f318a 
>   ql/src/test/results/clientpositive/ptf.q.out 2317347 
>   ql/src/test/results/clientpositive/ptf_streaming.q.out 427e635 
>   ql/src/test/results/clientpositive/smb_mapjoin_20.q.out 999dabd 
>   ql/src/test/results/clientpositive/smb_mapjoin_21.q.out 539b70e 
>   ql/src/test/results/clientpositive/spark/bucket2.q.out 5eb28fa 
>   ql/src/test/results/clientpositive/spark/bucket3.q.out 1b1010a 
>   ql/src/test/results/clientpositive/spark/bucket4.q.out 7dd49ac 
>   ql/src/test/results/clientpositive/spark/bucketsortoptimize_insert_2.q.out 365306e 
>   ql/src/test/results/clientpositive/spark/bucketsortoptimize_insert_4.q.out 3846de7 
>   ql/src/test/results/clientpositive/spark/bucketsortoptimize_insert_6.q.out 5b559c4 
>   ql/src/test/results/clientpositive/spark/bucketsortoptimize_insert_7.q.out cefc6aa 
>   ql/src/test/results/clientpositive/spark/bucketsortoptimize_insert_8.q.out ca44d7c 
>   ql/src/test/results/clientpositive/spark/disable_merge_for_bucketing.q.out 3864c44 
>   ql/src/test/results/clientpositive/spark/load_dyn_part2.q.out a8cef34 
>   ql/src/test/results/clientpositive/spark/ptf.q.out deebf3a 
>   ql/src/test/results/clientpositive/spark/ptf_streaming.q.out cd77c5f 
>   ql/src/test/results/clientpositive/spark/smb_mapjoin_20.q.out 9815d24 
>   ql/src/test/results/clientpositive/spark/smb_mapjoin_21.q.out 44bae6e 
>   ql/src/test/results/clientpositive/spark/stats10.q.out 4bec7fc 
>   ql/src/test/results/clientpositive/spark/vectorized_ptf.q.out eed14f3 
>   ql/src/test/results/clientpositive/stats10.q.out 804e1b8 
>   ql/src/test/results/clientpositive/tez/bucket2.q.out 9536ed0 
>   ql/src/test/results/clientpositive/tez/bucket3.q.out 3427474 
>   ql/src/test/results/clientpositive/tez/bucket4.q.out ca3584e 
>   ql/src/test/results/clientpositive/tez/disable_merge_for_bucketing.q.out dcffd87 
>   ql/src/test/results/clientpositive/tez/dynpart_sort_opt_vectorization.q.out 0d42286 
>   ql/src/test/results/clientpositive/tez/dynpart_sort_optimization.q.out ff44a57 
>   ql/src/test/results/clientpositive/tez/load_dyn_part2.q.out d1c4a10 
>   ql/src/test/results/clientpositive/tez/ptf.q.out 30ae91a 
>   ql/src/test/results/clientpositive/tez/ptf_streaming.q.out 9f35f22 
>   ql/src/test/results/clientpositive/tez/vector_bucket.q.out e4deb04 
>   ql/src/test/results/clientpositive/tez/vectorized_ptf.q.out d90aa63 
>   ql/src/test/results/clientpositive/vector_bucket.q.out 952c033 
>   ql/src/test/results/clientpositive/vectorized_ptf.q.out 75c8144 
> 
> Diff: https://reviews.apache.org/r/30151/diff/
> 
> 
> Testing
> -------
> 
> Existing tests.
> 
> 
> Thanks,
> 
> Ashutosh Chauhan
> 
>