o-nikolas commented on code in PR #56740:
URL: https://github.com/apache/airflow/pull/56740#discussion_r2437439721
##########
airflow-core/src/airflow/executors/executor_loader.py:
##########
@@ -211,7 +211,15 @@ def _get_team_executor_configs(cls) -> list[tuple[str |
None, list[str]]]:
executor_names = team_executor_config.strip("=")
else:
cls.block_use_of_multi_team()
- team_name, executor_names = team_executor_config.split("=")
+ if conf.getboolean("core", "multi_team", fallback=False):
Review Comment:
I chose to leave the other one in until we're ready for release. In my eyes
we'll have the `multi_team` config well into the future, so that Airflow is not
doing unnecessary multi-team actions unless it's enabled (to not affect the
default case). But until we're ready to release I'd like to leave in the HARD
stop of the dev block so that no one can even use it until we're ready.
What do you think?
--
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]
