github-actions[bot] opened a new pull request, #72160: URL: https://github.com/apache/airflow/pull/72160
BulkVariableService.handle_bulk_update and BulkPoolService.handle_bulk_update each ran one batched existence-check query up front, then discarded the result and re-queried every entity individually inside the loop. The sibling handle_bulk_delete methods in the same two files, and connections.py's handle_bulk_update, already reuse the batched-lookup dict correctly -- this was the one spot left over from the earlier bulk-delete N+1 fixes. update_orm_from_pydantic in both files now takes the already-fetched ORM object instead of a key/name string, matching the design connections.py already uses, so the update loop no longer needs its own per-item query. (cherry picked from commit 562e7307c2cffa1af1dc74813a215221d4c7ad9b) Co-authored-by: Jyun-An Chen <[email protected]> -- 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]
