o-nikolas commented on code in PR #38054:
URL: https://github.com/apache/airflow/pull/38054#discussion_r1525311303
##########
airflow/models/abstractoperator.py:
##########
@@ -58,6 +58,8 @@
DEFAULT_OWNER: str = conf.get_mandatory_value("operators", "default_owner")
DEFAULT_POOL_SLOTS: int = 1
DEFAULT_PRIORITY_WEIGHT: int = 1
+# TODO: Is None the right thing here? We want NULL essentially
Review Comment:
Good question, it's covered in more detail in the AIP (I wish I could hyper
link to one specific section or comment...) but essentially NULL, means that no
specific executor was requested for the task. In those cases we fall back to
the historical/default behaviour of Airflow, which is to use whatever the
currently configured environment level executor is, which may change at any
time (if the user decides to switch it).
--
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]
