potiuk commented on code in PR #30509:
URL: https://github.com/apache/airflow/pull/30509#discussion_r1160017797


##########
airflow/settings.py:
##########
@@ -605,3 +605,7 @@ def initialize():
 AIRFLOW_MOVED_TABLE_PREFIX = "_airflow_moved"
 
 DAEMON_UMASK: str = conf.get("core", "daemon_umask", fallback="0o077")
+
+# AIP-52: setup/teardown (experimental)
+# This feature is not complete yet, so we disable it by default.
+_ENABLE_AIP_52 = bool(os.environ.get("AIRFLOW_ENABLE_AIP_52", ""))

Review Comment:
   Setting "" as the falsy value of env var leads to some ambiguity. The above 
works more reasonably IMHO



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

Reply via email to