Skip to content

Make traceback for DB isolation mode way smaller and useful#41129

Merged
potiuk merged 1 commit intoapache:mainfrom
potiuk:remove-traceback-useless-frames
Jul 30, 2024
Merged

Make traceback for DB isolation mode way smaller and useful#41129
potiuk merged 1 commit intoapache:mainfrom
potiuk:remove-traceback-useless-frames

Conversation

@potiuk
Copy link
Copy Markdown
Member

@potiuk potiuk commented Jul 30, 2024

Removes a lot of useless frames from the traceback:

  • pytest and pluggy
  • last two "create_session" tracebacks

This way it is immediately visible where we should look for problems

Related: #41067


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

Removes a lot of useless frames from the traceback:

* pytest and pluggy
* last two "create_session" tracebacks

This way it is immediately visible where we should look for problems

Related: apache#41067
@potiuk
Copy link
Copy Markdown
Member Author

potiuk commented Jul 30, 2024

Only that remains:

image

@potiuk potiuk merged commit 7233ffe into apache:main Jul 30, 2024
@potiuk potiuk deleted the remove-traceback-useless-frames branch July 30, 2024 19:58
@utkarsharma2 utkarsharma2 added the changelog:skip Changes that should be skipped from the changelog (CI, tests, etc..) label Jul 31, 2024
@utkarsharma2 utkarsharma2 added this to the Airflow 2.10.0 milestone Jul 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog:skip Changes that should be skipped from the changelog (CI, tests, etc..)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants