amoghrajesh commented on PR #51873:
URL: https://github.com/apache/airflow/pull/51873#issuecomment-3036359447

   > The cause of this error was a change in the string representation of the 
exception after this PR was merged. In #52838, I changed it to a what seems 
like a more robust way of catching the "connection not found" exception. But I 
think at some point, we should probably make the connection getter raise 
exceptions in a more consistent manner (maybe once we fully switch over to Task 
SDK) so that callers of that method can have a simple try/except to handle the 
not found case
   
   Hey @ramitkataria what string representation changed? Could you elaborate a 
bit there?


-- 
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]

Reply via email to