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

Posted to commits@airflow.apache.org by "eladkal (via GitHub)" <gi...@apache.org> on 2023/12/01 04:40:04 UTC

Re: [PR] Avoid creating the hook in the EmrServerlessCancelJobsTrigger init [airflow]

eladkal commented on PR #35992:
URL: https://github.com/apache/airflow/pull/35992#issuecomment-1835447437

   > This might be considered a breaking change, but it's completely useless to create the hook in the init method because we create an instance from the trigger in the operator when we defer it, which means an extra needed DB query to fetch the secrets.
   
   I dont think this qualify as breaking change


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