[GitHub] [arrow] jorisvandenbossche commented on pull request #37822: GH-23221: [Python] python changes for pyodide build

["jorisvandenbossche (via GitHub)" <gi...@apache.org>]

jorisvandenbossche commented on PR #37822:
URL: https://github.com/apache/arrow/pull/37822#issuecomment-1733622941

   @joemarshall thanks for the PR!
   
   We might want to expose `is_threading_enabled()` in pyarrow publicly (it might be useful for downstream packages as well?), in `__init__.py` 
   
   > 2\. pyarrow sets defaults for inclusion of submodules based on their inclusion in the arrow build. e.g. pyarrow.parquet is built only if ARROW_PARQUET is set. This makes it possible to build in situations where you don't have access to set the build environment variables (e.g. in cross compiling situations like pyodide).
   
   This part is not actually included here? (or I don't understand the sentence) 
   (and it might also make sense to leave that for a third PR since it is changing the build process beyond emscriptem?)
   


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