-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/68648/
-----------------------------------------------------------
(Updated Sept. 7, 2018, 7:57 p.m.)
Review request for hive, Gopal V and Matt McCline.
Changes
-------
Missed non-vectorized case and some result updates.
Bugs: HIVE-20510
https://issues.apache.org/jira/browse/HIVE-20510
Repository: hive-git
Description
-------
Vectorization : Support loading bucketed tables using sorted dynamic partition
optimizer.
Added a new VectorExpression BucketNumberExpression to evaluate _bucket_number.
Made the loops as tight as possible.
Diffs (updated)
-----
ql/src/java/org/apache/hadoop/hive/ql/exec/FunctionRegistry.java 8bf0a9c77d
ql/src/java/org/apache/hadoop/hive/ql/exec/ReduceSinkOperator.java a2a9c8421e
ql/src/java/org/apache/hadoop/hive/ql/exec/vector/VectorizationContext.java
57f7c0108e
ql/src/java/org/apache/hadoop/hive/ql/exec/vector/expressions/BucketNumExpression.java
PRE-CREATION
ql/src/java/org/apache/hadoop/hive/ql/exec/vector/reducesink/VectorReduceSinkObjectHashOperator.java
5ab59c9c61
ql/src/java/org/apache/hadoop/hive/ql/optimizer/SortedDynPartitionOptimizer.java
51010aac85
ql/src/java/org/apache/hadoop/hive/ql/udf/generic/GenericUDFBucketNumber.java
PRE-CREATION
ql/src/test/queries/clientpositive/dynpart_sort_opt_vectorization.q
435cdaddd0
ql/src/test/results/clientpositive/llap/dynpart_sort_opt_vectorization.q.out
22f0a31eb3
ql/src/test/results/clientpositive/llap/dynpart_sort_optimization.q.out
21fc2c545a
ql/src/test/results/clientpositive/llap/dynpart_sort_optimization_acid.q.out
a0a5e0cf32
Diff: https://reviews.apache.org/r/68648/diff/3/
Changes: https://reviews.apache.org/r/68648/diff/2-3/
Testing
-------
Thanks,
Deepak Jaiswal
