Skip to content

Unpin pyarrow #17

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

Open
dhirschfeld opened this issue Feb 9, 2025 · 3 comments · May be fixed by #20
Open

Unpin pyarrow #17

dhirschfeld opened this issue Feb 9, 2025 · 3 comments · May be fixed by #20

Comments

@dhirschfeld
Copy link

The upper pin on pyarrow was removed in databricks-sql-python 3.7.0 which suggests it is also not required here:

Could the pin also be removed here as it makes it problematic to install alongside other libraries which also depend on pyarrow.

pyarrow = { version = ">=14.0.1,<17"}

@dhirschfeld
Copy link
Author

...and it also creates security issues if the pin isn't being constantly updated to the latest:

@dhirschfeld dhirschfeld linked a pull request Feb 18, 2025 that will close this issue
@euanmacinnes
Copy link

it also makes it problematic to install in a docker image with alpine as the prebuilt arrow is only 18.0.0 onwards. Better to have arrow as an optional pip install databricks-sqlalchemy[arrow]

@akoshel
Copy link

akoshel commented Mar 21, 2025

pyarrow makes this incompatible with databricks-sql-connector for python 3.13
https://github.com/databricks/databricks-sql-python/blob/v4.0.1/pyproject.toml#L25

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 a pull request may close this issue.

3 participants