You are viewing a plain text version of this content. The canonical link for it is here.

Posted to github@arrow.apache.org by GitBox <gi...@apache.org> on 2022/07/18 01:02:47 UTC

[GitHub] [arrow] wesm commented on pull request #12837: ARROW-15824: [Python] Make pyarrow.parquet a package

wesm commented on PR #12837:
URL: https://github.com/apache/arrow/pull/12837#issuecomment-1186654950

   I apologize for the bikeshed moment, but I was debugging something and found that much of pyarrow/parquet.py had been relocated to pyarrow/parquet/__init__.py -- I have often believed that having non-trivial implementation code in __init__.py files was an antipattern (and we have some code in pyarrow/__init__.py and maybe elsewhere that might be better moved to a utility module). However, my beliefs may not be aligned with broader community practices in the Python community nowadays. Thoughts from other frequent Python contributors (@jorisvandenbossche, @pitrou)? 


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