[GitHub] [druid] vtlim commented on a diff in pull request #13088: Add a note to the documentation about pre-built HLLSketches

[GitBox <gi...@apache.org>]

vtlim commented on code in PR #13088:
URL: https://github.com/apache/druid/pull/13088#discussion_r973368695


##########
docs/development/extensions-core/datasketches-hll.md:
##########
@@ -59,6 +59,11 @@ druid.extensions.loadList=["druid-datasketches"]
  }
 ```
 
+The `HLLSketchBuild` aggregator builds a datasketch from the input column specified. If used during ingestion, this
+will result in Druid storing pre-generated HLL Sketch objects in the datasource, rather than the original value itself.
+If used at query time on an existing dimension, the resulting column can be used as an intermediate dimension by the
+post-aggregators below.
+

Review Comment:
   I made a few minor edits to the suggestion above for consistency of language and style. One question: What does "original values" refer to? The raw data that made up the sketch?



-- 
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: commits-unsubscribe@druid.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org