ColtenOuO commented on PR #66222: URL: https://github.com/apache/airflow/pull/66222#issuecomment-4364654390
Thanks for the suggestion @jason810496 — I've added a test that uses `assert_queries_count` per the convention you linked. Additionally, while checking this unit test, I realized that using **round-trip** times provides a more accurate evaluation of the impact. I have updated the table below to reflect this: | Bulk size | Before | After | Saved | |---|---|---|---| | 10 pools | 14 | 4 | −71% | | 100 pools | 104 | 4 | −96% | If you have any feedback or suggestions for this PR, feel free to let me know! Thanks to all the reviewers for your support! -- 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]
