lazerg opened a new pull request, #76684:
URL: https://github.com/apache/airflow/pull/76684
<!-- SPDX-License-Identifier: Apache-2.0
https://www.apache.org/licenses/LICENSE-2.0 -->
related: #76683
`GET /api/v2/backfills` handed the lazy `session.scalars(...)` result
straight to `BackfillCollectionResponse.backfills`, which was typed
`Iterable[...]`. Pydantic validates an `Iterable` field lazily, so the rows
were fetched while FastAPI serialized the response on the event loop. The field
is now `list[BackfillResponse]` and the route passes `list(...)`, so the rows
are fetched in the threadpool with the rest of the route. `dag_model` was
already joined, so nothing else loads per item, and the UI route already passes
a list.
The OpenAPI schema is unchanged (both render as an array). The public and UI
backfill tests pass, and the guard test from #76537 reports this field as
`XPASS(strict)`, so its `backfills` line in
`generated/known_lazy_response_fields.txt` should go in whichever of the two
PRs merges second.
---
##### Was generative AI tooling used to co-author this PR?
- [X] Yes — Claude Code (Claude Opus 5.5)
Generated-by: Claude Code (Claude Opus 5.5) 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]
