Skip to content

Commit

Permalink
unpin pyarrow for windows (#42097)
Browse files Browse the repository at this point in the history
Toward fixing #38300

PR #29993 added a local ray fix for issue apache/arrow#26685, but at the time windows failed the tests with pyarrow7. In issue #38300 the suggested fix was to release the pin.

Signed-off-by: mattip <[email protected]>
Co-authored-by: Edward Oakes <[email protected]>
  • Loading branch information
mattip and edoakes authored Jan 25, 2024
1 parent 44db192 commit 68bd737
Showing 1 changed file with 1 addition and 5 deletions.
6 changes: 1 addition & 5 deletions python/setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -227,11 +227,7 @@ def get_packages(self):
if setup_spec.type == SetupType.RAY:
pandas_dep = "pandas >= 1.3"
numpy_dep = "numpy >= 1.20"
if sys.platform != "win32":
pyarrow_dep = "pyarrow >= 6.0.1"
else:
# Serialization workaround for pyarrow 7.0.0+ doesn't work for Windows.
pyarrow_dep = "pyarrow >= 6.0.1, < 7.0.0"
pyarrow_dep = "pyarrow >= 6.0.1"
setup_spec.extras = {
"data": [
numpy_dep,
Expand Down

0 comments on commit 68bd737

Please sign in to comment.