Skip to content

Add number of queries guard in public event logs list endpoints #57508

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 30, 2025
Merged

Add number of queries guard in public event logs list endpoints #57508

pierrejeambrun merged 1 commit into from
Oct 30, 2025

Conversation

@pierrejeambrun
Copy link
Member

@pierrejeambrun pierrejeambrun commented Oct 29, 2025

Add number of db queries guard in list endpoint, preventing further N+1 queries problem

N+1 queries problem detected and solved.

@pierrejeambrun pierrejeambrun added this to the Airflow 3.1.2 milestone Oct 29, 2025
@pierrejeambrun pierrejeambrun self-assigned this Oct 29, 2025
@pierrejeambrun pierrejeambrun added the backport-to-v3-1-test Mark PR with this label to backport to v3-1-test branch label Oct 29, 2025
@boring-cyborg boring-cyborg bot added the area:API Airflow's REST/HTTP API label Oct 29, 2025
@pierrejeambrun pierrejeambrun mentioned this pull request Oct 30, 2025
Merged
@pierrejeambrun pierrejeambrun force-pushed the add-query-guard-for-public-event-logs branch from abbe442 to ea4192d Compare October 30, 2025 12:57
guan404ming
guan404ming approved these changes Oct 30, 2025
View reviewed changes
Copy link
Member

@guan404ming guan404ming left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice!

@pierrejeambrun pierrejeambrun merged commit 2199c7a into apache:main Oct 30, 2025
114 checks passed
@pierrejeambrun pierrejeambrun deleted the add-query-guard-for-public-event-logs branch October 30, 2025 14:13
github-actions bot pushed a commit that referenced this pull request Oct 30, 2025
@pierrejeambrun
[v3-1-test] Add number of queries guard in public event logs list end…
795131e 
…points (#57508)

(cherry picked from commit 2199c7a)

Co-authored-by: Pierre Jeambrun <pierrejbrun@gmail.com>
@github-actions
Copy link

github-actions bot commented Oct 30, 2025

Backport successfully created: v3-1-test

Status Branch Result
✅ v3-1-test PR Link

pierrejeambrun added a commit that referenced this pull request Oct 31, 2025
@pierrejeambrun
[v3-1-test] Add number of queries guard in public event logs list end…
19b516f 
…points (#57508)

(cherry picked from commit 2199c7a)

Co-authored-by: Pierre Jeambrun <pierrejbrun@gmail.com>
pierrejeambrun added a commit that referenced this pull request Oct 31, 2025
@github-actions @pierrejeambrun
[v3-1-test] Add number of queries guard in public event logs list end…
297ef8a 
…points (#57508) (#57558)

(cherry picked from commit 2199c7a)

Co-authored-by: Pierre Jeambrun <pierrejbrun@gmail.com>
@ephraimbuddy ephraimbuddy added the type:bug-fix Changelog: Bug Fixes label Nov 10, 2025
Copilot AI pushed a commit to jason810496/airflow that referenced this pull request Dec 5, 2025
@pierrejeambrun
Add number of queries guard in public event logs list endpoints (apac…
94d4cfa 
…he#57508)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@guan404ming guan404ming guan404ming approved these changes

@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

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

Assignees

@pierrejeambrun pierrejeambrun

Labels

area:API Airflow's REST/HTTP API backport-to-v3-1-test Mark PR with this label to backport to v3-1-test branch type:bug-fix Changelog: Bug Fixes

Projects

None yet

Milestone

Airflow 3.1.2

Development

Successfully merging this pull request may close these issues.

3 participants

@pierrejeambrun @guan404ming @ephraimbuddy