jscheffl commented on code in PR #68198:
URL: https://github.com/apache/airflow/pull/68198#discussion_r3375881263
##########
airflow-core/src/airflow/settings.py:
##########
@@ -204,8 +204,10 @@ def dag_policy(dag):
return get_policy_plugin_manager().hook.dag_policy(dag=dag)
-def task_instance_mutation_hook(task_instance):
- return
get_policy_plugin_manager().hook.task_instance_mutation_hook(task_instance=task_instance)
+def task_instance_mutation_hook(task_instance, dag_run=None):
Review Comment:
Would it make sense - if we have our hand on it anyway - to match the
signature including typing?
```suggestion
def task_instance_mutation_hook(task_instance: TaskInstance, dag_run: DagRun
| None = None):
```
--
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]
