wip: relax upper pyarrow bound#428

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Jump to bottom

Open

rauerhans wants to merge1 commit intodatabricks:main

base:main

Choose a base branch

fromrauerhans:main

Open

wip: relax upper pyarrow bound#428

rauerhans wants to merge1 commit intodatabricks:mainfromrauerhans:main

Conversation

Copy link

rauerhans commentedAug 22, 2024

implements#427

wip: relax upper pyarrow bound

68a4b52

rauerhans requested review fromandrefurlan-db,benc-db,jackyhu-db,kravets-levko,rcypher-databricks andyunbodeng-db ascode owners

August 22, 2024 10:50

Copy link

Contributor

kravets-levko commentedAug 22, 2024

@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

Copy link

cpcloud commentedSep 20, 2024

Would be great to get this merged!

Copy link

Contributor

I see the fix here is to move topyarrow<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.