-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/71932/
-----------------------------------------------------------
(Updated Dec. 20, 2019, 3:21 p.m.)
Review request for hive and Jesús Camacho Rodríguez.
Bugs: HIVE-22652
https://issues.apache.org/jira/browse/HIVE-22652
Repository: hive-git
Description
-------
HIVE-22652: TopNKey push through Group by with Grouping sets
============================================================
Enable TNK op push down through Group by with Grouping sets by removing the
lines which checked whether the GBY operator has GROUPING SETS
Diffs (updated)
-----
ql/src/java/org/apache/hadoop/hive/ql/optimizer/topnkey/TopNKeyPushdownProcessor.java
c79c371a8b
ql/src/test/queries/clientpositive/topnkey_grouping_sets.q PRE-CREATION
ql/src/test/results/clientpositive/llap/topnkey_grouping_sets.q.out
PRE-CREATION
ql/src/test/results/clientpositive/llap/vector_groupby_grouping_sets_limit.q.out
c7e837905d
ql/src/test/results/clientpositive/perf/tez/cbo_query14.q.out d1e8c3806e
ql/src/test/results/clientpositive/perf/tez/cbo_query77.q.out aa080603e1
ql/src/test/results/clientpositive/perf/tez/constraints/cbo_query14.q.out
59fcf951fe
ql/src/test/results/clientpositive/perf/tez/constraints/cbo_query77.q.out
39da7ea903
ql/src/test/results/clientpositive/perf/tez/constraints/query14.q.out
65d3faa20f
ql/src/test/results/clientpositive/perf/tez/constraints/query27.q.out
e1a48eaeea
ql/src/test/results/clientpositive/perf/tez/constraints/query5.q.out
13288d28b4
ql/src/test/results/clientpositive/perf/tez/constraints/query77.q.out
c2758b7033
ql/src/test/results/clientpositive/perf/tez/constraints/query80.q.out
72a54928c2
ql/src/test/results/clientpositive/perf/tez/query14.q.out 00bc4cb026
ql/src/test/results/clientpositive/perf/tez/query27.q.out 774c0fd192
ql/src/test/results/clientpositive/perf/tez/query5.q.out 03980ac2c0
ql/src/test/results/clientpositive/perf/tez/query77.q.out fcfc5a33bc
ql/src/test/results/clientpositive/perf/tez/query80.q.out 3020b58781
Diff: https://reviews.apache.org/r/71932/diff/3/
Changes: https://reviews.apache.org/r/71932/diff/2-3/
Testing
-------
- New q test: topnkey_grouping_sets.q
- Run `src/test/queries/clientpositive/perf/query*.q` tests with
TestTezPerfCliDriver, TestTezPerfConstraintsCliDriver
Thanks,
Krisztian Kasa
