rok commented on a change in pull request #10476:
URL: https://github.com/apache/arrow/pull/10476#discussion_r651771596
##########
File path: cpp/src/arrow/compute/api_aggregate.h
##########
@@ -201,30 +201,42 @@ Result<Datum> MinMax(
/// \brief Test whether any element in a boolean array evaluates to true.
///
/// This function returns true if any of the elements in the array evaluates
-/// to true and false otherwise. Null values are skipped.
+/// to true and false otherwise. Null values are ignored by default.
+/// If null values are taken into account by setting ScalarAggregateOptions
+/// parameter skip_nulls = false then kleen logic is applied.
Review comment:
I think it KleeneOr. Added.
--
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]
