deniskuzZ commented on code in PR #6114:
URL: https://github.com/apache/hive/pull/6114#discussion_r2405656363
##########
ql/src/test/results/clientpositive/llap/vector_count.q.out:
##########
@@ -212,7 +212,7 @@ STAGE PLANS:
enabled: true
enabledConditionsMet:
hive.vectorized.use.vectorized.input.format IS true
inputFileFormats:
org.apache.hadoop.hive.ql.io.orc.OrcInputFormat
- notVectorizedReason: GROUPBY operator: Aggregations with > 1
parameter are not supported unless all the extra parameters are constants
count([Column[a], Column[b]])
+ notVectorizedReason: GROUPBY operator: Unsupported COUNT
DISTINCT with multiple columns: count([Column[a], Column[b]]). Hive only
supports COUNT(DISTINCT col) in vectorized execution.
Review Comment:
before we supported multi-column aggregations with constant expressions and
now we don't? At least that what the message was saying
````
Aggregations with > 1 parameter are not supported unless all the extra
parameters are constants count([Column[a], Column[b]])
````
i don't get why are we changing the message? if the issue was related to
filter on partition column, it shouldn't change non-partition table behavior
--
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]
