-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/71984/
-----------------------------------------------------------
Review request for hive and Jesús Camacho Rodríguez.
Bugs: HIVE-22666
https://issues.apache.org/jira/browse/HIVE-22666
Repository: hive-git
Description
-------
implement supporting partitioning in TopNKeyOperator
enable push down of partitioned TopNKeyOperator
Diffs
-----
ql/src/java/org/apache/hadoop/hive/ql/exec/TopNKeyOperator.java b7c1250220
ql/src/java/org/apache/hadoop/hive/ql/optimizer/topnkey/CommonKeyPrefix.java
049466050b
ql/src/java/org/apache/hadoop/hive/ql/optimizer/topnkey/TopNKeyProcessor.java
ce6efa4919
ql/src/java/org/apache/hadoop/hive/ql/optimizer/topnkey/TopNKeyPushdownProcessor.java
348fbb5faf
ql/src/java/org/apache/hadoop/hive/ql/plan/TopNKeyDesc.java a9e2a527df
ql/src/java/org/apache/hadoop/hive/ql/ppd/OpProcFactory.java 38f66f978c
ql/src/test/org/apache/hadoop/hive/ql/exec/TestTopNKeyFilter.java
PRE-CREATION
ql/src/test/org/apache/hadoop/hive/ql/optimizer/topnkey/TestCommonKeyPrefix.java
51d2382278
ql/src/test/queries/clientpositive/topnkey_windowing.q PRE-CREATION
ql/src/test/queries/clientpositive/topnkey_windowing_order.q PRE-CREATION
ql/src/test/queries/clientpositive/windowing_filter.q 2483c18416
ql/src/test/results/clientpositive/llap/topnkey.q.out 3150eefc7d
ql/src/test/results/clientpositive/llap/topnkey_grouping_sets.q.out
b1ec8d8847
ql/src/test/results/clientpositive/llap/topnkey_grouping_sets_functions.q.out
0ec6e6de43
ql/src/test/results/clientpositive/llap/topnkey_grouping_sets_order.q.out
d3ff1a6256
ql/src/test/results/clientpositive/llap/topnkey_windowing.q.out PRE-CREATION
ql/src/test/results/clientpositive/llap/topnkey_windowing_order.q.out
PRE-CREATION
ql/src/test/results/clientpositive/llap/windowing_filter.q.out 78240b58b3
ql/src/test/results/clientpositive/perf/tez/constraints/query49.q.out
37fc51697b
ql/src/test/results/clientpositive/perf/tez/constraints/query67.q.out
9ed7306c46
ql/src/test/results/clientpositive/perf/tez/constraints/query70.q.out
40dfaa2f13
ql/src/test/results/clientpositive/perf/tez/query49.q.out 239592bc1d
ql/src/test/results/clientpositive/perf/tez/query67.q.out 54d76e9c1a
ql/src/test/results/clientpositive/perf/tez/query70.q.out 23f9166a4f
Diff: https://reviews.apache.org/r/71984/diff/1/
Testing
-------
New q test were added
topnkey_windowing.q,topnkey_windowing_order.q
New UT:
TestTopNKeyFilter.java
existing:
windowing_filter.q,topnkey.q,topnkey_grouping_sets.q,topnkey_grouping_sets_functions.q,topnkey_grouping_sets_order.q
existing UT:
TestCommonKeyPrefix
Thanks,
Krisztian Kasa
