okumin commented on code in PR #5245:
URL: https://github.com/apache/hive/pull/5245#discussion_r1603378557
##########
ql/src/test/results/clientpositive/llap/count.q.out:
##########
@@ -811,7 +811,7 @@ STAGE PLANS:
Execution mode: llap
Reduce Operator Tree:
Group By Operator
- aggregations: count(DISTINCT KEY._col1:0._col0),
count(DISTINCT KEY._col1:1._col0), sum(VALUE._col0), sum(VALUE._col1),
sum(VALUE._col2), sum(KEY._col1:0._col0), sum(KEY._col1:1._col0),
sum(KEY._col0), sum(DISTINCT KEY._col1:2._col0), sum(DISTINCT KEY._col1:3._col0)
+ aggregations: count(DISTINCT KEY._col1:0._col0),
count(DISTINCT KEY._col1:1._col0), sum(VALUE._col0), sum(VALUE._col1),
sum(VALUE._col2), sum(KEY._col1:0._col0), sum(KEY._col1:1._col0),
sum(KEY._col1:2._col0), sum(DISTINCT KEY._col1:2._col0), sum(DISTINCT
KEY._col1:3._col0)
Review Comment:
`KEY._col0` had a special name because it is not only a distinct param but
also an aggregation param. So, it is accessed with two types of names in
GroupByOperator.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
