jorisvandenbossche commented on a change in pull request #11886:
URL: https://github.com/apache/arrow/pull/11886#discussion_r774510954
##########
File path: cpp/src/arrow/compute/kernels/vector_selection.cc
##########
@@ -2355,6 +2358,76 @@ const FunctionDoc array_take_doc(
"given by `indices`. Nulls in `indices` emit null in the output."),
{"array", "indices"}, "TakeOptions");
+const FunctionDoc indices_nonzero_doc(
+ "Return indices of the array containing non zero or false values",
Review comment:
I still find this summary line a bit confusing ("indices of the array"
sounds a bit strange, and the "non zero or false" isn't necessarily read as
"non-zero or non-false)" but rather as "non-zero or false)
Maybe something like: `Return the indices of the values in the array that
are non-zero` ?
--
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]
