Jackie-Jiang opened a new pull request #5805: URL: https://github.com/apache/incubator-pinot/pull/5805
## Description `AggregationFunction` itself is stateless, so we can share it among all the segments to prevent the overhead of creating it per segment. This can significantly improve the performance of high selectivity queries that hit lots of segments. Remove the `accept(visitor)` from the `AggregationFunction` interface which may make it stateful ## Release Notes Interface change: `accept(visitor)` is removed from `AggregationFunction` All the implementation of the `AggregationFunction` should be stateless so that it can be shared among all the segments. ---------------------------------------------------------------- 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]
