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


Reply via email to