ashb commented on a change in pull request #12184:
URL: https://github.com/apache/airflow/pull/12184#discussion_r522167065
##########
File path: airflow/models/dagbag.py
##########
@@ -47,6 +47,16 @@
from airflow.utils.session import provide_session
from airflow.utils.timeout import timeout
+# TODO: Remove once deprecated
+if hasattr(settings, "policy"):
+ warnings.warn(
+ "Using `policy` in airflow_local_settings.py is deprecated. "
+ "Please rename your `policy` to `task_policy`.",
+ DeprecationWarning,
+ stacklevel=2,
+ )
+ setattr(settings, "task_policy", settings.policy) # pylint:
disable=no-member
Review comment:
I'm just conscious that DagBag may go away at some point soon -- it does
a lot less than it used to.
Also top-level code is a bit of a smell to me.
----------------------------------------------------------------
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:
[email protected]
