This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

classification
Title: asyncio: cancelling wait_for(task, timeout) must also cancel the task
Type: Stage:
Components: asyncio Versions: Python 3.4, Python 3.5
process
Status: closed Resolution: fixed
Dependencies: Superseder:
Assigned To: Nosy List: gvanrossum, python-dev, vstinner, yselivanov
Priority: normal Keywords: patch

Created on 2015-01-10 10:02 by vstinner, last changed 2022-04-11 14:58 by admin. This issue is now closed.

Files
File name Uploaded Description Edit
cancel_wait_for-2.patch vstinner, 2015-01-10 10:08 review
Messages (4)
msg233813 - (view) Author: STINNER Victor (vstinner) * (Python committer) Date: 2015-01-10 10:02
Cancelling wait_for(task, timeout) must also cancel the waited task. Attached patch fixes this issue.

Original issue:
https://code.google.com/p/tulip/issues/detail?id=211
msg233814 - (view) Author: STINNER Victor (vstinner) * (Python committer) Date: 2015-01-10 10:08
Oops, I forgot to call remove_done_callback(). New patch attached.

My patch is based on Gustavo Carneiro's patch from Tulip issue #211.
msg234080 - (view) Author: Roundup Robot (python-dev) (Python triager) Date: 2015-01-15 15:30
New changeset 8adf1896712d by Victor Stinner in branch '3.4':
Closes #23219: cancelling asyncio.wait_for() now cancels the task
https://hg.python.org/cpython/rev/8adf1896712d
msg240000 - (view) Author: Roundup Robot (python-dev) (Python triager) Date: 2015-04-03 15:10
New changeset c167b9f9aac8 by Victor Stinner in branch '3.4':
Issue #23219: Update asyncio.wait_for() documentation
https://hg.python.org/cpython/rev/c167b9f9aac8
History
Date User Action Args
2022-04-11 14:58:11adminsetgithub: 67408
2015-04-03 15:10:34python-devsetmessages: + msg240000
2015-01-15 15:31:44vstinnersetstage: resolved ->
2015-01-15 15:30:40python-devsetstatus: open -> closed

nosy: + python-dev
messages: + msg234080

resolution: fixed
stage: resolved
2015-01-10 10:08:38vstinnersetfiles: + cancel_wait_for-2.patch

messages: + msg233814
2015-01-10 10:03:49vstinnersetfiles: - cancel_wait_for.patch
2015-01-10 10:02:13vstinnercreate