yashmayya commented on code in PR #17519:
URL: https://github.com/apache/pinot/pull/17519#discussion_r2733793306
##########
pinot-broker/src/main/java/org/apache/pinot/broker/requesthandler/BaseSingleStageBrokerRequestHandler.java:
##########
@@ -159,7 +159,14 @@ public abstract class BaseSingleStageBrokerRequestHandler
extends BaseBrokerRequ
Map.entry("sum", "summv"),
Map.entry("minmaxrange", "minmaxrangemv"),
Map.entry("distinctcounthllplus", "distinctcounthllplusmv"),
- Map.entry("distinctcountrawhllplus", "distinctcountrawhllplusmv")
+ Map.entry("distinctcountrawhllplus", "distinctcountrawhllplusmv"),
Review Comment:
Yeah all the overrides will be removed after the next release when all of
these aggregations will be consolidated. This is just adding some of the
currently missing ones that will remain only until the next release. I can
revert it if you'd prefer though.
--
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]
