Skip to content

Conversation

@henry3260
Copy link
Contributor

closes: #59899


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

@henry3260
Copy link
Contributor Author

I will fix the CI tests.

@henry3260 henry3260 force-pushed the add-dependencies-sql branch from 302e996 to 311b097 Compare December 30, 2025 20:11
@potiuk
Copy link
Member

potiuk commented Dec 31, 2025

It's a bit more than that - see merged PRs linked to similar issues in parent.

@vurgunhajiyev
Copy link

Hi! I’d like to help complete this issue. I see there is a PR #59945 opened.
Could you please clarify what the intended dependency changes are for common.sql
and if there are tests I should add for them?

@vurgunhajiyev
Copy link

Thanks for the PR! I reviewed the change and it looks correct to me.
Adding sqlalchemy>=1.4.49 to providers/common/sql dependencies
fully addresses the issue. No additional tests seem necessary here.

@henry3260
Copy link
Contributor Author

henry3260 commented Dec 31, 2025

Thanks for the PR! I reviewed the change and it looks correct to me.

Adding sqlalchemy>=1.4.49 to providers/common/sql dependencies

fully addresses the issue. No additional tests seem necessary here.

Yeah, I already have a fix in this pr. I will commit later :)!

@henry3260 henry3260 force-pushed the add-dependencies-sql branch from 311b097 to 0fcf840 Compare January 1, 2026 12:30
@potiuk potiuk merged commit 90b2472 into apache:main Jan 1, 2026
91 checks passed
@henry3260 henry3260 deleted the add-dependencies-sql branch January 1, 2026 12:55
henry3260 added a commit to henry3260/airflow that referenced this pull request Jan 3, 2026
stegololz pushed a commit to stegololz/airflow that referenced this pull request Jan 9, 2026
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.

Add proper dependencies for sqlalchemy - common.sql

3 participants