jason810496 commented on PR #76416: URL: https://github.com/apache/airflow/pull/76416#issuecomment-6628794866
> From what I see, each API worker now opens a second (async) pool with the same size settings as `[database] sql_alchemy_pool_size` / `max_overflow`, so it can use up to twice as many DB connections, right? Should we mention that it PR desc atleast? This this the second route using async DB session, the first one is https://github.com/apache/airflow/pull/67800, let me check whether the 3.4.0 release note include this part or not. > Ash's test gap thread is resolved, but I only see the `assert_awaited_once` mock test. Is the concurrency regression test coming in #76424, or should we open a separate issue for it? It was addressed in https://github.com/apache/airflow/pull/76416/changes/8e190039499a94581d9b2183dbf78d7b95fd0f22 commit, let's me comment in the thread as well to avoid the confusion. -- 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]
