Consolidate UI Dependency Graphs#69833
Open
bbovenzi wants to merge 5 commits into
Open
Conversation
boring-cyborg
Bot
added
area:API
bbovenzi
marked this pull request as ready for review
July 14, 2026 18:53
bbovenzi
requested review from
bugraoz93,
choo121600,
ephraimbuddy,
guan404ming,
henry3260,
jason810496,
pierrejeambrun,
rawwar,
ryanahamilton,
shubhamraj-git and
vatsrahul1001
as code owners
July 14, 2026 18:53
bbovenzi
force-pushed
the
simple-dep-graph
branch
2 times, most recently
from
July 15, 2026 16:12
6a55a54 to
c01c322
Compare
Member
|
Just one small question, otherwise looking good to me. (needs rebase though) |
…ies graph Resolving each scheduled Dag's entry task previously deserialized the full Dag once per Dag discovered while tracing an asset's dependencies -- for an asset feeding many Dags, that's N full Dag deserializations plus N round trips. Restructures the BFS to process a whole frontier of assets per round and batch-resolve every scheduled Dag's entry point in a single query.
bbovenzi
force-pushed
the
simple-dep-graph
branch
from
July 21, 2026 13:48
c01c322 to
e4aab14
Compare
Fix nested boolean asset conditions dropping sibling branches so (a & b) | (c & d) renders both gates; read a scheduled Dag's entry task from serialized JSON instead of deserializing every Dag while tracing an asset's lineage; and recover alias-produced assets' source task from AssetEvent so they still show a producer. Also harden a malformed expression against a 500 and link a Dag to every triggering asset in the degraded fallback.
The dependency-view toggle moved from a per-Dag localStorage key to a single global one, leaving the old `dependencies-<dag_id>` entries stranded in every user's browser storage. Clean them up once at load.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Before, users had to toggle between "immediate dependencies" and "all dependencies" options. Immediate dependencies showed the asset conditions but only one level above or below the dag in question. "All dependencies" didn't include asset conditions. This was always meant to be temporary.
This PR consolidates the two options into one endpoint. And always includes the logic gates for Any/all conditions.
Also, localstorage now saves the "show all dag dependencies" or not globally so it is easier to navigate between dags and assets without the graph losing connections.
Was generative AI tooling used to co-author this PR?
{pr_number}.significant.rst, in airflow-core/newsfragments. You can add this file in a follow-up commit after the PR is created so you know the PR number.