FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Add a workaround for bpo-37658 by elprans · Pull Request #608 · MagicStack/asyncpg · GitHub

Add a workaround for bpo-37658 - #608

Merged
elprans merged 1 commit into
masterfrom
workaround-bpo37658
Aug 27, 2020
Merged

Add a workaround for bpo-37658#608
elprans merged 1 commit into
masterfrom
workaround-bpo37658

Conversation

elprans commented Aug 16, 2020
edited
Loading

Copy link
Copy Markdown
Member

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 git@aliddell.com
Fixes: #467
Fixes: #547
Related: #468
Supersedes: #548

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

Copy link
Copy Markdown

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 <git@aliddell.com>
Fixes: #467
Fixes: #547
Related: #468
Supersedes: #548
elprans force-pushed the workaround-bpo37658 branch from 3f516c5 to 2bac166 Compare August 27, 2020 16:53
elprans merged commit 2bac166 into master Aug 27, 2020

Copy link
Copy Markdown
Contributor

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

elprans commented Oct 23, 2020

Copy link
Copy Markdown
Member Author

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

elprans deleted the workaround-bpo37658 branch October 23, 2020 16:22
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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


Back | FazBrowse Home | New Git URL