jihoonson commented on pull request #10518: URL: https://github.com/apache/druid/pull/10518#issuecomment-736112042
One more comment. I think the query should be rewritten before it is sent to historicals to not include `GroupingAggregatorFactory` because subtotalsSpec is currently computed only in brokers. You probably want to modify [these lines](https://github.com/apache/druid/blob/master/processing/src/main/java/org/apache/druid/query/groupby/GroupByQueryQueryToolChest.java#L232-L257). ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
