raulcd commented on code in PR #47626:
URL: https://github.com/apache/arrow/pull/47626#discussion_r2373246296
##########
ci/scripts/python_wheel_unix_test.sh:
##########
@@ -101,8 +102,10 @@ if [ "${CHECK_WHEEL_CONTENT}" == "ON" ]; then
fi
if [ "${CHECK_UNITTESTS}" == "ON" ]; then
- # Install testing dependencies
- python -m pip install -U -r
"${source_dir}/python/requirements-wheel-test.txt"
+ if [ "${FREE_THREADED}" == "OFF" ]; then
+ # Install testing dependencies
+ python -m pip install -U -r
"${source_dir}/python/requirements-wheel-test.txt"
Review Comment:
Thanks @AlenkaF this is great. Should we remove the requirements
installation on the docker file and install based on the new `FREE_THREADED`
flag here?
I mean, remove the:
`RUN python -m pip install --no-build-isolation -r
/arrow/python/requirements-wheel-test-3.13t.txt`
and install it on the `python_wheel_unix_test.sh` when `"${FREE_THREADED}"
== "ON"` so we install requirements for both free-threaded and non
free-threaded on the same place
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
