uros-b commented on code in PR #57551:
URL: https://github.com/apache/spark/pull/57551#discussion_r3681404059


##########
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/aggregate/kllAggregates.scala:
##########
@@ -208,6 +208,13 @@ case class KllSketchAggBigint(
       The optional k parameter controls the size and accuracy of the sketch 
(default 200, range 8-65535).
       Larger k values provide more accurate quantile estimates but result in 
larger, slower sketches.
   """,
+  arguments = """
+    Arguments:
+      * expr - The expression to aggregate into the KLL sketch.
+        An expression that evaluates to a float or double.

Review Comment:
   so only float here? no double



-- 
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]

Reply via email to