Copilot commented on code in PR #70903:
URL: https://github.com/apache/airflow/pull/70903#discussion_r3695736084
##########
airflow-ctl/src/airflowctl/api/client.py:
##########
@@ -517,8 +516,6 @@ def get_client(
kind=kind,
)
yield api_client
- except AirflowCtlNotFoundException as e:
- raise e
finally:
if api_client:
Review Comment:
The removed `except AirflowCtlNotFoundException as e: raise e` block was not
a strict no-op: `raise e` resets the traceback to this frame, while letting the
exception propagate naturally preserves the original traceback from where it
was raised. This is likely an improvement, but it means the PR description’s
claim of “no change in behavior / identically to before” isn’t accurate (stack
traces/debugging output will differ).
--
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]
