You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2022/08/02 03:00:44 UTC

[GitHub] [airflow] uranusjr commented on pull request #25430: Fix common sql DbApiHook fetch_all_handler

uranusjr commented on PR #25430:
URL: https://github.com/apache/airflow/pull/25430#issuecomment-1201959797

   I think the problem with `try .. except` is less about the performance (the difference should be minimal compared to the actual SQL database connection time), but we don’t know what to catch in the first place :p Otherwise I’d use that.


-- 
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-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org