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 2020/07/21 20:38:34 UTC

[GitHub] [airflow] waleedsamy edited a comment on pull request #9910: fix task_instance_mutation_hook

waleedsamy edited a comment on pull request #9910:
URL: https://github.com/apache/airflow/pull/9910#issuecomment-662092184


   This issue is happening with **1.10.11** - not related to master branch.
   Looking into [v1.10-stable](https://github.com/apache/airflow/blob/d23fa2f8896c5e31745131e6917f1970b4aa590f/airflow/__init__.py#L48) airflow/\_\_init\_\_.py
   
   ```py
   from airflow import utils
   from airflow import settings
   from airflow.configuration import conf
   from airflow.models import DAG
   from flask_admin import BaseView
   from importlib import import_module
   from airflow.exceptions import AirflowException
   
   settings.initialize()
   ```
   
   it will do a bunch of imports before calling initialize. These import will reach `dagrun.py` and `from airflow.settings import task_instance_mutation_hook` will cache `task_instance_mutation_hook` so even when calling `settings.initialize()` it won't affect the `task_instance_mutation_hook` in `dagrun.py` anymore.


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

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