tlm365 commented on PR #43602: URL: https://github.com/apache/arrow/pull/43602#issuecomment-2273113405
@jorisvandenbossche Could you take a look please, I choose use `importlib.resources()` (1) instead of `numpy.get_include()` (2) because: (2) need `import numpy`, then some current release test will fail (eg: [ci/scripts/release_test.sh](https://github.com/apache/arrow/actions/runs/10281234832/job/28450333148)), we need to add `numpy` as a dependency for `requirements-test` (1) failed Python 3.8 CI pipeline (importlib.resources.files available from >=3.9). But, this Python version planning was removed since Arrow v18.0.0, so I think (1) makes more sense. How do you think about it? -- 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
