----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/68648/ -----------------------------------------------------------
(Updated Sept. 8, 2018, 8:38 a.m.) Review request for hive, Gopal V and Matt McCline. Changes ------- Updated results for failing tests. 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) ----- itests/hive-blobstore/src/test/results/clientpositive/insert_into_dynamic_partitions.q.out 74a9a56f07 itests/hive-blobstore/src/test/results/clientpositive/insert_overwrite_dynamic_partitions.q.out ee02c36f03 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/dynpart_sort_optimization_acid2.q.out aea757205f 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 ql/src/test/results/clientpositive/show_functions.q.out 90608e2905 Diff: https://reviews.apache.org/r/68648/diff/4/ Changes: https://reviews.apache.org/r/68648/diff/3-4/ Testing ------- Thanks, Deepak Jaiswal