pierrejeambrun opened a new pull request, #65966:
URL: https://github.com/apache/airflow/pull/65966
Backport of #65775 to `v3-2-test`.
Cherry-picked from be41cb4031e38050314c9c9605885edee5dd1df0.
Conflict in
`airflow-core/src/airflow/api_fastapi/core_api/routes/ui/grid.py`:
- v3-2-test still uses the simpler `_build_ti_summaries` shape
(`collections.defaultdict(list)` + `_get_aggs_for_node`, returning `dict`) —
kept that, just switched the DAG lookup to the shared `DBDagBag`.
- v3-2-test's `DBDagBag` does not yet expose a public `get_dag(version_id,
session)` (added on main alongside the LRU/TTL cache). Used `_get_dag(...)`
instead, matching how
`airflow.api_fastapi.common.dagbag.get_dag_for_run_or_latest_version` already
calls it on this branch.
- Narrowed the `dag_version_id` parameter to `UUID | None` in `_get_serdag`
to match v3-2-test's `DBDagBag._get_dag` signature (mypy caught this).
- v3-2-test's `_find_aggregates` yields plain dicts (not `(node, _)`
tuples), so kept `for node in _find_aggregates(...)`.
- Dropped the per-request `serdag_cache: dict = {}` in
`get_grid_ti_summaries_stream` — the shared `DBDagBag` instance attached to
`app.state` already deduplicates across runs and across requests.
Generated files (`_private_ui.yaml`, the openapi-gen `*.ts`) auto-merged
cleanly.
---
##### Was generative AI tooling used to co-author this PR?
- [X] Yes — Claude Code (Opus 4.7)
Generated-by: Claude Code (Opus 4.7) following [the
guidelines](https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#gen-ai-assisted-contributions)
--
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]
