[GitHub] [arrow] amol- commented on a change in pull request #11886: ARROW-13035: [C++] indices_nonzero compute function

[GitBox <gi...@apache.org>]

amol- commented on a change in pull request #11886:
URL: https://github.com/apache/arrow/pull/11886#discussion_r777539373



##########
File path: docs/source/cpp/compute.rst
##########
@@ -1549,6 +1549,17 @@ These functions select and return a subset of their input.
 * \(4) For each element *i* in input 2 (the indices), the *i*'th element
   in input 1 (the values) is appended to the output.
 
+Containment tests
+~~~~~~~~~~~~~~~~~
+
+This functions return the indices at which elements match a predicate
+
++-----------------------+-------+-----------------------------------+----------------+---------------------------------+-------+
+| Function name         | Arity | Input types                       | Output type    | Options class                   | Notes |

Review comment:
       I thought about this initially, but I think that keeping those two columns will make life easier for additional compute functions that might have notes or options. Given that indentation in rst tables makes easy to mess them up, already preparing the columns will make life easier for future additions.




-- 
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-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org