amol- commented on a change in pull request #11624: URL: https://github.com/apache/arrow/pull/11624#discussion_r752345889
########## File path: python/pyarrow/compute.py ########## @@ -70,6 +70,7 @@ function_registry, get_function, list_functions, + _group_by Review comment: It's to make it available when `_pc()` is used to access compute functions from other modules/files (in this case from `table.pxi`). I adhered to that practice instead of injecting a `import pyarrow._compute`. Given that there are many more internal functions in the `pyarrow.compute` module I thought it wasn't concerning. -- 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-unsubscr...@arrow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org