-
-
Notifications
You must be signed in to change notification settings - Fork 6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Run sync context managers in a single thread #5114
Conversation
Codecov Report
@@ Coverage Diff @@
## master #5114 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 532 533 +1
Lines 13672 13716 +44
=========================================
+ Hits 13672 13716 +44
Continue to review full report at Codecov.
|
📝 Docs preview for commit 732c0e9 at: https://62c8571bf399773d59da1a9b--fastapi.netlify.app |
📝 Docs preview for commit c2d73cd at: https://62c8582b4d7df6396e3dd6a6--fastapi.netlify.app |
📝 Docs preview for commit e39c97c at: https://62c866fbf77cbd4b372cd172--fastapi.netlify.app |
…into unlimited-power
📝 Docs preview for commit d6202f5 at: https://62c8680e69e7074ec8565f20--fastapi.netlify.app |
Closing in favor of #5122 |
This avoids a deadlock where
__exit__
can't be called because there are no thread tokens available. Partially resolves #3205 and other related issues.