Skip to content

[v3-1-test] Fix n+1 queries in ui/dags endpoint and use a compact response model. (#57425) #57444

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
pierrejeambrun merged 1 commit into from
Oct 28, 2025
Merged

[v3-1-test] Fix n+1 queries in ui/dags endpoint and use a compact response model. (#57425) #57444

pierrejeambrun merged 1 commit into from
Oct 28, 2025

Conversation

@github-actions
Copy link

@github-actions github-actions bot commented Oct 28, 2025

  • Use DAGRunLightResponse to stop loading unused fields in response that trigger n+1 queries to other tables.

  • Update frontend type and tests.

  • Fix MyPy error.
    (cherry picked from commit f722f3c)

Co-authored-by: Karthikeyan Singaravelan tir.karthi@gmail.com

@tirkarthi
[v3-1-test] Fix n+1 queries in ui/dags endpoint and use a compact res…
d21cc14 
…ponse model. (#57425)

* Use DAGRunLightResponse to stop loading unused fields in response that trigger n+1 queries to other tables.

* Update frontend type and tests.

* Fix MyPy error.
(cherry picked from commit f722f3c)

Co-authored-by: Karthikeyan Singaravelan <tir.karthi@gmail.com>
@github-actions github-actions bot mentioned this pull request Oct 28, 2025
Merged
@boring-cyborg boring-cyborg bot added area:API Airflow's REST/HTTP API area:UI Related to UI/UX. For Frontend Developers. labels Oct 28, 2025
@pierrejeambrun pierrejeambrun marked this pull request as ready for review October 28, 2025 15:57
@pierrejeambrun pierrejeambrun merged commit 7787564 into v3-1-test Oct 28, 2025
82 checks passed
@pierrejeambrun pierrejeambrun deleted the backport-f722f3c-v3-1-test branch October 28, 2025 17:19
@kaxil kaxil mentioned this pull request Oct 31, 2025
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@bbovenzi bbovenzi Awaiting requested review from bbovenzi bbovenzi is a code owner

@pierrejeambrun pierrejeambrun Awaiting requested review from pierrejeambrun pierrejeambrun is a code owner

@ryanahamilton ryanahamilton Awaiting requested review from ryanahamilton ryanahamilton is a code owner

@jscheffl jscheffl Awaiting requested review from jscheffl jscheffl is a code owner

@shubhamraj-git shubhamraj-git Awaiting requested review from shubhamraj-git shubhamraj-git is a code owner

@ephraimbuddy ephraimbuddy Awaiting requested review from ephraimbuddy ephraimbuddy is a code owner

@rawwar rawwar Awaiting requested review from rawwar rawwar is a code owner

@jason810496 jason810496 Awaiting requested review from jason810496 jason810496 is a code owner

@bugraoz93 bugraoz93 Awaiting requested review from bugraoz93 bugraoz93 is a code owner

Assignees

No one assigned

Labels

area:API Airflow's REST/HTTP API area:UI Related to UI/UX. For Frontend Developers.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@pierrejeambrun @tirkarthi