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 2021/12/08 13:08:14 UTC

[GitHub] [airflow] potiuk edited a comment on pull request #20133: Applying the TI mutation hook to tasks run from UI

potiuk edited a comment on pull request #20133:
URL: https://github.com/apache/airflow/pull/20133#issuecomment-988799452


   Yep. Agree with @ashb . It should not be possible to run the mutation hook in here. 
   
   First of all, what would be the purpose of it ? 
   
   Secondly if you look closely the task_instance_mutation hook should definitely not be called here:
   
   https://airflow.apache.org/docs/apache-airflow/stable/concepts/cluster-policies.html
   
   | Takes a TaskInstance parameter called task_instance. **Called right before task execution.**


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