arw2019 commented on a change in pull request #8294:
URL: https://github.com/apache/arrow/pull/8294#discussion_r504128151
##########
File path: cpp/src/arrow/compute/api_aggregate.h
##########
@@ -142,6 +142,20 @@ Result<Datum> MinMax(const Datum& value,
const MinMaxOptions& options = MinMaxOptions::Defaults(),
ExecContext* ctx = NULLPTR);
+/// \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 taken to evaluate to false.
Review comment:
Docstring updated.
Sorry yes. The examples were for `all`. Just to make sure I understand, with
kleene logic, we emit null if there's a null anywhere in the input
```
any_kleene([true, null]) = null
any_kleene([false, null]) = null
all_kleene([true, null]) = null
all_kleene([false, null]) = null
```
I opened ARROW-10301 re: `all` kernel
----------------------------------------------------------------
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]
