Skip to content

Comments

handle SQLAlchemyError via app exception handler#62320

Open
henry3260 wants to merge 1 commit intoapache:mainfrom
henry3260:replace-to-fastapi-custom-exception
Open

handle SQLAlchemyError via app exception handler#62320
henry3260 wants to merge 1 commit intoapache:mainfrom
henry3260:replace-to-fastapi-custom-exception

Conversation

@henry3260
Copy link
Contributor

@henry3260 henry3260 commented Feb 22, 2026 •
edited
Loading

Changes

Added a centralized SQLAlchemyError exception handler in app.py, and remove try-except

Was generative AI tooling used to co-author this PR?
  • Yes (please specify the tool below)
    Copilot
  • Read the Pull Request Guidelines for more information. Note: commit author/co-author name and email in commits become permanently public when merged.
  • For fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
  • When adding dependency, check compliance with the ASF 3rd Party License Policy.
  • For significant user-facing changes create newsfragment: {pr_number}.significant.rst or {issue_number}.significant.rst, in airflow-core/newsfragments.

@henry3260 henry3260 force-pushed the replace-to-fastapi-custom-exception branch from 4112682 to 1b7646e Compare February 22, 2026 13:23
@henry3260 henry3260 force-pushed the replace-to-fastapi-custom-exception branch from 1b7646e to 196c09f Compare February 22, 2026 14:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ashb ashb Awaiting requested review from ashb ashb is a code owner

@kaxil kaxil Awaiting requested review from kaxil kaxil is a code owner

@amoghrajesh amoghrajesh Awaiting requested review from amoghrajesh amoghrajesh is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@henry3260