chore: Loosen pyarrow#5661
Conversation
franciscojavierarceo
left a comment
There was a problem hiding this comment.
can you update setup.py and then run make install-python-dependencies-dev?
Signed-off-by: dlovell <[email protected]>
Signed-off-by: dlovell <[email protected]>
4c06ac4 to
d56c8f6
Compare
|
hmm looks like macos-14 doesn't like it @ntkathole can you provide some guidance here? |
|
i'm not sure i created my venv with i'll start over with a venv created that way |
|
looks like |
The requirement file included in PR has nvidia-* dependencies which is only meant for GPU supported systems. You need to make sure |
|
Also, fyi - we have APACHE_ARROW_VERSION="17.0.0" added in feast-operator images, which needs to be updated as well. |
3777db6 to
7a46e80
Compare
|
@ntkathole FYI: i've modified its not clear that |
Signed-off-by: dlovell <[email protected]>
7a46e80 to
af98a25
Compare
Looks like it is appending
I see, right. No need to update since it's still pin to 17.0.0 |
Signed-off-by: dlovell <[email protected]>
af98a25 to
e8e4b95
Compare
|
@ntkathole FYI: this branch also includes a change to the ray / codeflare dependency environment marker. |
What this PR does / why we need it:
This PR loosens the pyarrow requirement to allow for more recent pyarrow versions
Which issue(s) this PR fixes:
fixes #5609