clintropolis commented on code in PR #14497: URL: https://github.com/apache/druid/pull/14497#discussion_r1247209626
########## docs/querying/aggregations.md: ########## @@ -422,6 +389,121 @@ It is not possible to determine a priori how well this aggregator will behave fo For these reasons, we have deprecated this aggregator and recommend using the DataSketches Quantiles aggregator instead for new and existing use cases, although we will continue to support Approximate Histogram for backwards compatibility. + +## Expression aggregators + +### Expression aggregator + +Query time only aggregator that can aggregate results using [Druid expressions](./math-expr.md) functions to facilitate building custom functions. Review Comment: i don't really understand, there are several real examples of the expression aggregator after the table, and the table seems a more suitable place to explain the syntax rather than a pseudo json example like the javascript aggregator uses... -- 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]
