jon-wei commented on a change in pull request #7784: Add finalizeAsBase64Binary option to FixedBucketsHistogramAggregatorFactory URL: https://github.com/apache/incubator-druid/pull/7784#discussion_r292714899
########## File path: docs/content/development/extensions-core/approximate-histograms.md ########## @@ -124,6 +124,7 @@ For general histogram and quantile use cases, the [DataSketches Quantiles Sketch |`upperLimit`|Upper limit of the histogram. |No default, must be specified| |`numBuckets`|Number of buckets for the histogram. The range [lowerLimit, upperLimit] will be divided into `numBuckets` intervals of equal size.|10| |`outlierHandlingMode`|Specifies how values outside of [lowerLimit, upperLimit] will be handled. Supported modes are "ignore", "overflow", and "clip". See [outlier handling modes](#outlier-handling-modes) for more details.|No default, must be specified| +|`finalizeAsBase64Binary`|If true, the finalized aggregator value will be a Base64-encoded byte array containing the [serialized form](#serialization-formats) of the histogram. If false, the finalized aggregator value will be a JSON representation of the histogram.|false| Review comment: The ApproximateHistogram in the same extension currently has a human-readable finalized form, so I decided to keep this false for consistency. I also added the same property to ApproximateHistogramFactory ---------------------------------------------------------------- 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] With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
