nooneuse commented on code in PR #63143:
URL: https://github.com/apache/doris/pull/63143#discussion_r3258701351
##########
fe/fe-catalog/src/main/java/org/apache/doris/catalog/AggregateType.java:
##########
@@ -48,6 +49,7 @@ public enum AggregateType {
aggTypeMap.put("MAX", AggregateType.MAX);
aggTypeMap.put("REPLACE", AggregateType.REPLACE);
aggTypeMap.put("REPLACE_IF_NOT_NULL",
AggregateType.REPLACE_IF_NOT_NULL);
+ aggTypeMap.put("DATASKETCHES_HLL_UNION_AGG",
AggregateType.DATASKETCHES_HLL_UNION_AGG);
Review Comment:
You're right, there's really no need to add this. I initially added this
aggregation type because I wanted to support pre-aggregate, but it turns out
that's not necessary for now. I'll remove this aggregation type.
--
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]
