Skip to content

Add a workaround for bpo-37658 #608

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

Merged
merged 1 commit into from
Aug 27, 2020
Merged

Add a workaround for bpo-37658 #608

merged 1 commit into from
Aug 27, 2020

Conversation

elprans
Copy link
Member

@elprans elprans commented Aug 16, 2020

asyncio.wait_for() currently has a bug where it raises a
CancelledError even when the wrapped awaitable has completed.
The upstream fix is in python/cpython#21894. This adds a workaround
until the aforementioned PR is merged, backported and released.

Co-authored-by: Adam Liddell [email protected]
Fixes: #467
Fixes: #547
Related: #468
Supersedes: #548

@elprans elprans requested a review from 1st1 August 16, 2020 00:09
@elprans elprans force-pushed the workaround-bpo37658 branch 2 times, most recently from 278c684 to 3f516c5 Compare August 16, 2020 00:15
@yenbao1340
Copy link

Upstream's bug was fixed and merged (python/cpython#21894).
How can I apply this patch in my project?

`asyncio.wait_for()` currently has a bug where it raises a
`CancelledError` even when the wrapped awaitable has completed.
The upstream fix is in python/cpython#21894.  This adds a workaround
until the aforementioned PR is merged, backported and released.

Co-authored-by: Adam Liddell <[email protected]>
Fixes: #467
Fixes: #547
Related: #468
Supersedes: #548
@elprans elprans force-pushed the workaround-bpo37658 branch from 3f516c5 to 2bac166 Compare August 27, 2020 16:53
@elprans elprans merged commit 2bac166 into master Aug 27, 2020
@aaliddell
Copy link
Contributor

Could this make it into a patch release, if a minor/major isn't planned any time soon?

@elprans
Copy link
Member Author

elprans commented Oct 23, 2020

Yeah, it's a good time to cut a release, since both Python 3.9 and PostgreSQL 13 have been released recently.

@elprans elprans deleted the workaround-bpo37658 branch October 23, 2020 16:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Pool never closes if acquire with timeout is cancelled Leaked connections caused by timeout on pool.acquire
4 participants