Skip to content
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

Refactor Sqlalchemy queries to 2.0 style (Part 4) #32339

Merged
merged 3 commits into from
Jul 7, 2023

Conversation

phanikumv
Copy link
Contributor

This is a continuation of the effort to refactor the queries to sqlalchemy 2.0

related: #28723


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

@boring-cyborg boring-cyborg bot added area:secrets area:webserver Webserver related Issues labels Jul 4, 2023
@phanikumv phanikumv closed this Jul 4, 2023
@phanikumv phanikumv reopened this Jul 4, 2023
@potiuk potiuk added the full tests needed We need to run full set of tests for this PR to merge label Jul 4, 2023
@potiuk
Copy link
Member

potiuk commented Jul 4, 2023

applied "full tests needed" to that one as well.

@phanikumv phanikumv force-pushed the sqla_utils branch 2 times, most recently from 47e394f to 9d0255a Compare July 6, 2023 10:57
@phanikumv phanikumv marked this pull request as ready for review July 6, 2023 10:59
@phanikumv phanikumv merged commit 26bb408 into apache:main Jul 7, 2023
@phanikumv phanikumv deleted the sqla_utils branch July 7, 2023 11:22
syedahsn pushed a commit to aws-mwaa/upstream-to-airflow that referenced this pull request Jul 11, 2023
@ephraimbuddy ephraimbuddy added changelog:skip Changes that should be skipped from the changelog (CI, tests, etc..) and removed changelog:skip Changes that should be skipped from the changelog (CI, tests, etc..) labels Aug 2, 2023
@ephraimbuddy ephraimbuddy added the type:misc/internal Changelog: Misc changes that should appear in change log label Aug 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:secrets area:webserver Webserver related Issues full tests needed We need to run full set of tests for this PR to merge type:misc/internal Changelog: Misc changes that should appear in change log
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants