Skip to content

[v3-0-test] Add sqlalchemy-spanner limitation for broken 1.12.0 release #51432

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

Merged
merged 1 commit into from
Jun 5, 2025

Conversation

Lee-W
Copy link
Member

@Lee-W Lee-W commented Jun 5, 2025


^ 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.

…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
@Lee-W
Copy link
Member Author

Lee-W commented Jun 5, 2025

@potiuk I think we probably need to backport #51393 as well?

@Lee-W
Copy link
Member Author

Lee-W commented Jun 5, 2025

during backporting #51431, i encountered this issue

@vincbeck
Copy link
Contributor

vincbeck commented Jun 5, 2025

It should not be necessary given #51433?

@potiuk
Copy link
Member

potiuk commented Jun 5, 2025

It should not be necessary given #51433?

probably better to cherry-pick both -> this way there will be less conflicts if we need to cherry-pick another one in the future

@potiuk potiuk changed the title Add sqlalchemy-spanner limitation for broken 1.12.0 release [v3-0-test] Add sqlalchemy-spanner limitation for broken 1.12.0 release Jun 5, 2025
@potiuk potiuk merged commit 19221c8 into apache:v3-0-test Jun 5, 2025
2 checks passed
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]>
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