Skip to content

SQLA2: fix mypy issue with getting the dialect name#56941

Merged
vincbeck merged 2 commits intoapache:mainfrom
Dev-iL:2510/sqla2_dialect_retrieval
Oct 21, 2025
Merged

SQLA2: fix mypy issue with getting the dialect name#56941
vincbeck merged 2 commits intoapache:mainfrom
Dev-iL:2510/sqla2_dialect_retrieval

Conversation

@Dev-iL
Copy link
Collaborator

@Dev-iL Dev-iL commented Oct 21, 2025

Fix the 8 instances of the error below across airflow-core

error: Item "None" of "Engine | Connection | None" has no attribute "dialect"  [union-attr]

related: #56735


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

@Dev-iL
Copy link
Collaborator Author

Dev-iL commented Oct 21, 2025

The failure is unrelated to the changes, and will be fixed by #56944

Copy link
Member

@jason810496 jason810496 left a comment

Choose a reason for hiding this comment

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

Nice! Thanks for the PR!

@Dev-iL Dev-iL force-pushed the 2510/sqla2_dialect_retrieval branch 2 times, most recently from 05e3e7a to c4b3238 Compare October 21, 2025 17:57
@vincbeck vincbeck added the all versions If set, the CI build will be forced to use all versions of Python/K8S/DBs label Oct 21, 2025
@vincbeck vincbeck closed this Oct 21, 2025
@vincbeck vincbeck reopened this Oct 21, 2025
@Dev-iL Dev-iL force-pushed the 2510/sqla2_dialect_retrieval branch from c4b3238 to a41665c Compare October 21, 2025 18:13
@vincbeck vincbeck merged commit d7b440b into apache:main Oct 21, 2025
375 of 378 checks passed
@Dev-iL Dev-iL deleted the 2510/sqla2_dialect_retrieval branch October 22, 2025 05:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

all versions If set, the CI build will be forced to use all versions of Python/K8S/DBs area:API Airflow's REST/HTTP API area:DAG-processing area:serialization area:Triggerer

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants