ramitkataria commented on code in PR #50515:
URL: https://github.com/apache/airflow/pull/50515#discussion_r2101370215
##########
providers/amazon/src/airflow/providers/amazon/aws/hooks/base_aws.py:
##########
@@ -603,12 +607,22 @@ def conn_config(self) -> AwsConnectionWrapper:
"""Get the Airflow Connection object and wrap it in helper (cached)."""
connection = None
if self.aws_conn_id:
+ possible_exceptions = (
+ (AirflowNotFoundException, AirflowRuntimeError)
Review Comment:
Yea I think we can resolve and merge this for now and discuss on slack
instead? If we end up going with the approach I proposed, then the changes in
this PR wouldn't be needed anymore and instead, the change would be in sdk code
--
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: commits-unsubscr...@airflow.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
