Skip to content

Add sqlalchemy-spanner limitation for broken 1.12.0 release #51393

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

Conversation

potiuk
Copy link
Member

@potiuk potiuk commented Jun 4, 2025

When constraints are calculated, we should termporarily exclude the sqlalchemy-spanner from the install command until they fix the problem where 1.12.0 was relesed without .whl files.

Tracked in googleapis/python-spanner-sqlalchemy#682


^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named {pr_number}.significant.rst or {issue_number}.significant.rst, in airflow-core/newsfragments.

Copy link
Contributor

@amoghrajesh amoghrajesh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Brilliant!

Copy link
Contributor

@eladkal eladkal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cool!

@potiuk potiuk force-pushed the exclude-sqlalchemy-spanner-for-highest-resolution-in-pypi branch 2 times, most recently from c143ad6 to 401f81f Compare June 4, 2025 09:07
When constraints are calculated, we should termporarily exclude
the sqlalchemy-spanner from the install command until they fix
the problem where 1.12.0 was relesed without .whl files.

Tracked in googleapis/python-spanner-sqlalchemy#682
@potiuk potiuk force-pushed the exclude-sqlalchemy-spanner-for-highest-resolution-in-pypi branch from 401f81f to 1109384 Compare June 4, 2025 09:08
@potiuk potiuk merged commit 5193cb9 into apache:main Jun 4, 2025
94 checks passed
@potiuk potiuk deleted the exclude-sqlalchemy-spanner-for-highest-resolution-in-pypi branch June 4, 2025 09:49
Lee-W pushed a commit to astronomer/airflow that referenced this pull request Jun 5, 2025
…1393)

When constraints are calculated, we should termporarily exclude
the sqlalchemy-spanner from the install command until they fix
the problem where 1.12.0 was relesed without .whl files.

Tracked in googleapis/python-spanner-sqlalchemy#682
potiuk added a commit that referenced this pull request Jun 5, 2025
…se (#51393) (#51432)

When constraints are calculated, we should termporarily exclude
the sqlalchemy-spanner from the install command until they fix
the problem where 1.12.0 was relesed without .whl files.

Tracked in googleapis/python-spanner-sqlalchemy#682

Co-authored-by: Jarek Potiuk <[email protected]>
kaxil pushed a commit that referenced this pull request Jun 6, 2025
…se (#51393) (#51432)

When constraints are calculated, we should termporarily exclude
the sqlalchemy-spanner from the install command until they fix
the problem where 1.12.0 was relesed without .whl files.

Tracked in googleapis/python-spanner-sqlalchemy#682

Co-authored-by: Jarek Potiuk <[email protected]>
sanederchik pushed a commit to sanederchik/airflow that referenced this pull request Jun 7, 2025
…1393)

When constraints are calculated, we should termporarily exclude
the sqlalchemy-spanner from the install command until they fix
the problem where 1.12.0 was relesed without .whl files.

Tracked in googleapis/python-spanner-sqlalchemy#682
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants