amrishlal opened a new pull request #6323: URL: https://github.com/apache/incubator-pinot/pull/6323
This PR adds support for specifying decimal percentile (99.999, 0.00001, etc) through the following functions: ``` PERCENTILE(column, percentile) PERCENTILEEST(column, percentile) PERCENTILETDIGEST(column, percentile) PERCENTILEMV(column, percentile) PERCENTILEESTMV(column, percentile) PERCENTILETDIGESTMV(column, percentile) ``` Note that there is no change in the external behavior of existing percentile functions as described in [documentation](https://docs.pinot.apache.org/users/user-guide-query/supported-aggregations). However, the internal implementation of these functions were changed to support decimal percentile values as well. Modified to allow for successfully parsing and validating decimal values in new percentile functions: > AggregationFunctionFactory.java Existing implementation of percentile functions were modified to allow for supporting both the existing and the new percentile functions with decimal values: > PercentileAggregationFunction.java > PercentileEstAggregationFunction.java > PercentileEstMVAggregationFunction.java > PercentileMVAggregationFunction.java > PercentileTDigestAggregationFunction.java > PercentileTDigestMVAggregationFunction.java Modified to add new unit tests: > AggregationFunctionFactoryTest.java > PercentileTDigestQueriesTest.java I will update the documentation once this PR is complete. ---------------------------------------------------------------- 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]
