jason810496 opened a new pull request, #76416:
URL: https://github.com/apache/airflow/pull/76416

   related: 
https://github.com/apache/airflow/issues/75865#issuecomment-6622866800
   
   The audit log dependency is async but wrote its row through the route's sync 
session, so under concurrent write requests it waited for a pooled connection 
on the event loop, while the connections it waited for could only be released 
by that same loop. The API server then stalled until `pool_timeout` and 
returned 500s. With 24 concurrent `POST /api/v2/variables` on 3.4.0rc2 and 
Postgres 16, 403 of 600 requests finished in 180s, with 16 failing; with this 
change all 600 finish in 3.8s.
   
   The audit row is now written on its own async session, so no database call 
in that dependency blocks the event loop. This is a 3.4.0 RC regression, so it 
should be backported to `v3-4-test`.
   
   ---
   
   ##### Was generative AI tooling used to co-author this PR?
   
   - [X] Yes (please specify the tool below)
   
   Generated-by: Claude Code (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]

Reply via email to