Skip to content

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

Merged
phanikumv merged 3 commits into
apache:mainfrom
astronomer:sqla_utils
Jul 7, 2023
Merged

Refactor Sqlalchemy queries to 2.0 style (Part 4)#32339
phanikumv merged 3 commits into
apache:mainfrom
astronomer:sqla_utils

Conversation

@phanikumv

Copy link
Copy Markdown
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

potiuk commented Jul 4, 2023

Copy link
Copy Markdown
Member

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
Comment thread tests/cli/commands/test_connection_command.py Outdated
@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