pierrejeambrun commented on PR #44187: URL: https://github.com/apache/airflow/pull/44187#issuecomment-2486086281
If we have more concurrent request in FastAPI than connections allowed in the Pool, we might run into a problem indeed. But we can argue that this would already be the case with thread scoped session. FastAPI handles sync request in different thread so multiple concurrent request end up using multiple threads and therefore I think the same occurs. I don't think that removing a thread local registry for session directly impact how connection pooling is done. -- 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]
