Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

wip: relax upper pyarrow bound #428

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

rauerhans
Copy link

implements #427

@kravets-levko
Copy link
Contributor

@rauerhans Thank you for submitting this! I see you marked it as "WIP" - please let me know when you're done with this change, so I can review and merge

@cpcloud
Copy link

cpcloud commented Sep 20, 2024

Would be great to get this merged!

@arredond
Copy link
Contributor

arredond commented Oct 4, 2024

I see the fix here is to move to pyarrow<18.0 as the upper bound. Would you consider not adding an upper bound at all? Upper bounds really make dependency management much, much harder for complex projects.

Here's a nice discussion on the topic.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants