You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@hive.apache.org by Hari Sankar Sivarama Subramaniyan <hs...@hortonworks.com> on 2014/08/07 23:54:07 UTC

Review Request 24474: HIVE-6959 Enable Constant propagation optimizer for Hive Vectorization

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

Review request for hive, Ashutosh Chauhan and Jitendra Pandey.


Bugs: HIVE-6959
    https://issues.apache.org/jira/browse/HIVE-6959


Repository: hive-git


Description
-------

Enable Constant propagation optimizer for Hive Vectorization


Diffs
-----

  ql/src/java/org/apache/hadoop/hive/ql/exec/vector/VectorizationContext.java 535e4b3 
  ql/src/java/org/apache/hadoop/hive/ql/exec/vector/expressions/ConstantVectorExpression.java 9fd3853 
  ql/src/java/org/apache/hadoop/hive/ql/exec/vector/expressions/VectorExpressionWriterFactory.java eeb76d7 
  ql/src/java/org/apache/hadoop/hive/ql/optimizer/ConstantPropagate.java b12d3a8 
  ql/src/java/org/apache/hadoop/hive/ql/optimizer/physical/Vectorizer.java 50ce5b8 
  ql/src/test/org/apache/hadoop/hive/ql/exec/vector/TestVectorizationContext.java 2329f52 
  ql/src/test/queries/clientpositive/vector_coalesce.q 052ab71 
  ql/src/test/results/clientpositive/vector_between_in.q.out 78e340b 
  ql/src/test/results/clientpositive/vector_cast_constant.q.out cdb13cb 
  ql/src/test/results/clientpositive/vector_coalesce.q.out 9561d47 
  ql/src/test/results/clientpositive/vector_decimal_mapjoin.q.out 71a3def 
  ql/src/test/results/clientpositive/vector_decimal_math_funcs.q.out 717e81a 
  ql/src/test/results/clientpositive/vector_elt.q.out ea0af62 
  ql/src/test/results/clientpositive/vectorization_14.q.out 3992bb1 
  ql/src/test/results/clientpositive/vectorization_15.q.out 1f48fea 
  ql/src/test/results/clientpositive/vectorization_16.q.out 38596e6 
  ql/src/test/results/clientpositive/vectorization_9.q.out c757b1f 
  ql/src/test/results/clientpositive/vectorization_div0.q.out b2321b4 
  ql/src/test/results/clientpositive/vectorization_short_regress.q.out 5b23850 
  ql/src/test/results/clientpositive/vectorized_math_funcs.q.out 181ab51 
  ql/src/test/results/clientpositive/vectorized_parquet.q.out 2e459a8 

Diff: https://reviews.apache.org/r/24474/diff/


Testing
-------


Thanks,

Hari Sankar Sivarama Subramaniyan


Re: Review Request 24474: HIVE-6959 Enable Constant propagation optimizer for Hive Vectorization

Posted by Hari Sankar Sivarama Subramaniyan <hs...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/24474/
-----------------------------------------------------------

(Updated Aug. 8, 2014, 7:16 p.m.)


Review request for hive, Ashutosh Chauhan and Jitendra Pandey.


Bugs: HIVE-6959
    https://issues.apache.org/jira/browse/HIVE-6959


Repository: hive-git


Description
-------

Enable Constant propagation optimizer for Hive Vectorization


Diffs (updated)
-----

  ql/src/java/org/apache/hadoop/hive/ql/exec/vector/VectorizationContext.java 535e4b3 
  ql/src/java/org/apache/hadoop/hive/ql/exec/vector/expressions/ConstantVectorExpression.java 9fd3853 
  ql/src/java/org/apache/hadoop/hive/ql/exec/vector/expressions/VectorExpressionWriterFactory.java eeb76d7 
  ql/src/java/org/apache/hadoop/hive/ql/optimizer/ConstantPropagate.java b12d3a8 
  ql/src/java/org/apache/hadoop/hive/ql/optimizer/physical/Vectorizer.java e778ba4 
  ql/src/test/org/apache/hadoop/hive/ql/exec/vector/TestVectorizationContext.java 2329f52 
  ql/src/test/queries/clientpositive/vector_coalesce.q 052ab71 
  ql/src/test/results/clientpositive/tez/vector_cast_constant.q.out 9dac17b 
  ql/src/test/results/clientpositive/tez/vectorization_14.q.out 04c99f1 
  ql/src/test/results/clientpositive/tez/vectorization_15.q.out 1381695 
  ql/src/test/results/clientpositive/tez/vectorization_9.q.out 3d2645a 
  ql/src/test/results/clientpositive/tez/vectorization_short_regress.q.out 2fa1bae 
  ql/src/test/results/clientpositive/vector_between_in.q.out 78e340b 
  ql/src/test/results/clientpositive/vector_cast_constant.q.out cdb13cb 
  ql/src/test/results/clientpositive/vector_coalesce.q.out 9561d47 
  ql/src/test/results/clientpositive/vector_decimal_mapjoin.q.out 71a3def 
  ql/src/test/results/clientpositive/vector_decimal_math_funcs.q.out 717e81a 
  ql/src/test/results/clientpositive/vector_elt.q.out ea0af62 
  ql/src/test/results/clientpositive/vectorization_14.q.out 3992bb1 
  ql/src/test/results/clientpositive/vectorization_15.q.out 1f48fea 
  ql/src/test/results/clientpositive/vectorization_16.q.out 38596e6 
  ql/src/test/results/clientpositive/vectorization_9.q.out c757b1f 
  ql/src/test/results/clientpositive/vectorization_div0.q.out b2321b4 
  ql/src/test/results/clientpositive/vectorization_short_regress.q.out 5b23850 
  ql/src/test/results/clientpositive/vectorized_math_funcs.q.out 181ab51 
  ql/src/test/results/clientpositive/vectorized_parquet.q.out 2e459a8 

Diff: https://reviews.apache.org/r/24474/diff/


Testing
-------


Thanks,

Hari Sankar Sivarama Subramaniyan