[GitHub] [arrow] edponce commented on a change in pull request #11237: ARROW-14012: [Python] Update kernel categories in compute doc to match C++

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

edponce commented on a change in pull request #11237:
URL: https://github.com/apache/arrow/pull/11237#discussion_r717960772



##########
File path: docs/source/python/api/compute.rst
##########
@@ -388,18 +467,9 @@ Structural Transforms
 .. autosummary::
    :toctree: ../generated/
 
-   binary_length
-   case_when
-   choose
-   coalesce
-   fill_null
-   if_else
-   is_finite
-   is_inf
-   is_nan
-   is_null
-   is_valid
    list_element
    list_flatten
    list_parent_indices
    list_value_length
+   make_struct
+   replace_with_mask

Review comment:
       I agree. The reason for these changes is because in C++ docs categorize kernels based on their kind (aggregate, scalar, vector) but Python docs do not. In C++, `make_struct` is listed as a scalar structural transform and `replace_with_mask` as a special case of vector structural transforms](https://github.com/apache/arrow/pull/11237/files#diff-ce5b94577014735990903d3d03bd4ea4b8c8e6d32f5227592e60b7dd6a912d59R1544). In Python docs, I place them under the unified structural transform category (`replace_with_mask` was missing).




-- 
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