Skip to content

Conversation

@o-nikolas
Copy link
Contributor

This blocks the migration for adding the Teams table, because the executor loader will try to verify that teams exist before the table is added.

Also, whether teams exist or not is not relevant when compiling the list of cli commands, causing excess latency with DB queries.


^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named {pr_number}.significant.rst or {issue_number}.significant.rst, in airflow-core/newsfragments.

This blocks the migration for adding the Teams table, because the
executor loader will try to verify that teams exist before the table is
added.

Also, whether teams exist or not is not relevant when compiling the list
of cli commands, causing excess latency with DB queries.
@boring-cyborg boring-cyborg bot added area:CLI area:Executors-core LocalExecutor & SequentialExecutor labels Nov 8, 2025
@o-nikolas o-nikolas changed the title Don't verify team existence during cli parser loading AIP-67 Multi Team: Don't verify team existence during cli parser loading Nov 19, 2025
@o-nikolas o-nikolas requested a review from potiuk November 19, 2025 17:03
@o-nikolas o-nikolas added the multi-team - aip-67 Issues related to multi-team (AIP-67) label Nov 20, 2025
@potiuk potiuk merged commit 2d16ba4 into apache:main Nov 24, 2025
64 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:CLI area:Executors-core LocalExecutor & SequentialExecutor multi-team - aip-67 Issues related to multi-team (AIP-67)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants