-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/72013/
-----------------------------------------------------------
(Updated Jan. 17, 2020, 12:49 p.m.)
Review request for hive and Jesús Camacho Rodríguez.
Bugs: HIVE-22735
https://issues.apache.org/jira/browse/HIVE-22735
Repository: hive-git
Description
-------
TopNKey operator deduplication
==============================
When creating a TopNKeyDesc ignore `ExprNodeConstantDesc` type columns
Diffs (updated)
-----
ql/src/java/org/apache/hadoop/hive/ql/plan/TopNKeyDesc.java a9e2a527df
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/vector_groupby_grouping_sets_limit.q.out
3addfb9a88
ql/src/test/results/clientpositive/perf/tez/constraints/query14.q.out
44da18ad40
ql/src/test/results/clientpositive/perf/tez/constraints/query27.q.out
4eb41168bc
ql/src/test/results/clientpositive/perf/tez/constraints/query5.q.out
f78b2faf7c
ql/src/test/results/clientpositive/perf/tez/constraints/query77.q.out
40810d45dc
ql/src/test/results/clientpositive/perf/tez/constraints/query80.q.out
462eed8942
ql/src/test/results/clientpositive/perf/tez/query14.q.out da6b74d147
ql/src/test/results/clientpositive/perf/tez/query27.q.out 62fbeda71e
ql/src/test/results/clientpositive/perf/tez/query5.q.out 34bbb51ef9
ql/src/test/results/clientpositive/perf/tez/query77.q.out 6b9c5b4e16
ql/src/test/results/clientpositive/perf/tez/query80.q.out 30f77de916
Diff: https://reviews.apache.org/r/72013/diff/2/
Changes: https://reviews.apache.org/r/72013/diff/1-2/
Testing
-------
mvn test -Dtest.output.overwrite -DskipSparkTests
-Dtest=TestMiniLlapLocalCliDriver
-Dqfile=topnkey.q,topnkey_dedup.q,topnkey_grouping_sets_functions.q,topnkey_grouping_sets_order.q,topnkey_order_null.q,vector_topnkey.q
-pl itests/qtest -Pitests
Thanks,
Krisztian Kasa
