potiuk commented on PR #24915: URL: https://github.com/apache/airflow/pull/24915#issuecomment-2102002480
> Before this change, exceptions thrown by paramiko (eg. host not reachable, etc) were caught and raised forward as an AirflowException. After this change, that's not the case. Was that intentional @hsnprsd? If not, I can raise a separate issue if you agree this is not desired behaviour. Airlfow Exception is only relevant as base exception for some special behaviours (timeout etc.). There is no particular reason why you should **expect** provider to throw only Airflow Exception, at any point in time any unhandled exception can be raised so your code should be ready to handle any of those. -- 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]
