pitrou commented on a change in pull request #10887:
URL: https://github.com/apache/arrow/pull/10887#discussion_r689625509
##########
File path: docs/source/cpp/compute.rst
##########
@@ -230,10 +234,98 @@ Notes:
Note that the output can have less than *N* elements if the input has
less than *N* distinct values.
+ The mode kernel is not a proper aggregate (it is actually a vector
+ function, see below).
Review comment:
Is it important to mention this here?
##########
File path: docs/source/cpp/compute.rst
##########
@@ -230,10 +234,98 @@ Notes:
Note that the output can have less than *N* elements if the input has
less than *N* distinct values.
+ The mode kernel is not a proper aggregate (it is actually a vector
+ function, see below).
+
* \(5) Output is Int64, UInt64 or Float64, depending on the input type.
* \(6) Output is Float64 or input type, depending on QuantileOptions.
+ The quantile kernel is not a proper aggregate (it is actually a vector
+ function, see below).
Review comment:
Same question here.
--
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: github-unsubscr...@arrow.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
