Vamsi-klu commented on PR #66296: URL: https://github.com/apache/airflow/pull/66296#issuecomment-4676926247
Addressed the review and rebased on main: - `session.rollback()` is now wrapped in `suppress(Exception)`, with a unit test pinning that the original delete error propagates when rollback itself raises. - Added a code comment explaining why the archive-table drop runs on the session's own connection. - Newsfragment removed. On dag_version being FK-pinned by task_instance: this PR only converts the hang into a fast, visible failure — #68339 is the right place for skipping pinned rows. To keep the two PRs composable, the two backend regression tests here now disable row-exclusion config on a copied `_TableConfig` dict, so they keep forcing the FK-failure path that pins the hang regression even after #68339 lands. --- Drafted-by: Claude Code (Fable 5) (no human review before posting) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
