Skip to content

bpo-34679: ProactorEventLoop only uses set_wakeup_fd() in main thread#16901

Merged
vstinner merged 1 commit into
python:masterfrom
vstinner:proactor_set_wakeup_fd
Oct 23, 2019
Merged

bpo-34679: ProactorEventLoop only uses set_wakeup_fd() in main thread#16901
vstinner merged 1 commit into
python:masterfrom
vstinner:proactor_set_wakeup_fd

Conversation

@vstinner

@vstinner vstinner commented Oct 23, 2019

Copy link
Copy Markdown
Member

bpo-34679, bpo-38563: asyncio.ProactorEventLoop.close() now only calls
signal.set_wakeup_fd() in the main thread.

https://bugs.python.org/issue34679

bpo-34679, bpo-38563: asyncio.ProactorEventLoop.close() now only calls
signal.set_wakeup_fd() in the main thread.
@vstinner

Copy link
Copy Markdown
Member Author

I didn't test my PR manually on Windows, but I added a test which should test it on the CI.

@vstinner
vstinner merged commit 1b53a24 into python:master Oct 23, 2019
@vstinner
vstinner deleted the proactor_set_wakeup_fd branch October 23, 2019 15:25
@miss-islington

Copy link
Copy Markdown
Contributor

Thanks @vstinner for the PR 🌮🎉.. I'm working now to backport this PR to: 3.8.
🐍🍒⛏🤖

@miss-islington

Copy link
Copy Markdown
Contributor

I'm having trouble backporting to 3.8. Reason: 'Error 110 while writing to socket. Connection timed out.'. Please retry by removing and re-adding the needs backport to 3.8 label.

@miss-islington

Copy link
Copy Markdown
Contributor

Thanks @vstinner for the PR 🌮🎉.. I'm working now to backport this PR to: 3.8.
🐍🍒⛏🤖 I'm not a witch! I'm not a witch!

@bedevere-bot

Copy link
Copy Markdown

GH-16902 is a backport of this pull request to the 3.8 branch.

miss-islington added a commit that referenced this pull request Oct 23, 2019
…GH-16901)

bpo-34679, bpo-38563: asyncio.ProactorEventLoop.close() now only calls
signal.set_wakeup_fd() in the main thread.
(cherry picked from commit 1b53a24)

Co-authored-by: Victor Stinner <vstinner@python.org>
jacobneiltaylor pushed a commit to jacobneiltaylor/cpython that referenced this pull request Dec 5, 2019
…pythonGH-16901)

bpo-34679, bpo-38563: asyncio.ProactorEventLoop.close() now only calls
signal.set_wakeup_fd() in the main thread.
shihai1991 pushed a commit to shihai1991/cpython that referenced this pull request Jan 31, 2020
…pythonGH-16901)

bpo-34679, bpo-38563: asyncio.ProactorEventLoop.close() now only calls
signal.set_wakeup_fd() in the main thread.
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.

6 participants