[ 
https://issues.apache.org/jira/browse/ARROW-7871?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17066229#comment-17066229
 ] 

Wes McKinney edited comment on ARROW-7871 at 3/24/20, 10:21 PM:
----------------------------------------------------------------

> You cannot use the pyarrow.compute module in the Array methods as this would 
> introduce a cyclic dependency but you can directly call the C++ methods there.

I don't think this would be an issue. Since we explicitly add symbols to 
{{pyarrow.*}} in pyarrow/__init__.py we could do the same thing in 
pyarrow/compute.py. This would spare code duplication by having a single point 
of contact with each C++ kernel


was (Author: wesmckinn):
> You cannot use the pyarrow.compute module in the Array methods as this would 
> introduce a cyclic dependency but you can directly call the C++ methods there.

I don't think this would be an issue. Since we explicitly add symbols to 
{{pyarrow.*}} in pyarrow/__init__.py we could do the same thing in 
pyarrow/compute.py

> [Python] Expose more compute kernels
> ------------------------------------
>
>                 Key: ARROW-7871
>                 URL: https://issues.apache.org/jira/browse/ARROW-7871
>             Project: Apache Arrow
>          Issue Type: Improvement
>          Components: Python
>            Reporter: Krisztian Szucs
>            Priority: Major
>
> Currently only the sum kernel is exposed.
> Or consider to deprecate/remove the pyarrow.compute module, and bind the 
> compute kernels as methods instead.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to