lidavidm commented on a change in pull request #10887:
URL: https://github.com/apache/arrow/pull/10887#discussion_r689629822
##########
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:
I added it since it would come up eventually if you used the scalar
aggregation node through bindings, as you wouldn't be able to call it from
there, despite being listed as an "aggregate".
--
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
