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.
Created on 2014年04月28日 18:05 by qmega, last changed 2022年04月11日 14:58 by admin. This issue is now closed.
| Messages (5) | |||
|---|---|---|---|
| msg217390 - (view) | Author: Philip Sequeira (qmega) | Date: 2014年04月28日 18:05 | |
Example: https://docs.python.org/3.4/library/asyncio-task.html TimeoutError is mentioned several times, and links to the OSError subclass. However, the actual TimeoutError raised by asyncio stuff is the one from concurrent.futures, which is not compatible. The docs as they are seem to suggest that something like "except TimeoutError" would be appropriate, when in fact that would not produce the expected behavior; "except asyncio.TimeoutError" is what you'd want. |
|||
| msg217400 - (view) | Author: Antoine Pitrou (pitrou) * (Python committer) | Date: 2014年04月28日 19:20 | |
Gasp. Perhaps concurrent.futures.TimeoutError can inherit from the standard TimeoutError? The following patch doesn't seem to disrupt the test suite: diff --git a/Lib/concurrent/futures/_base.py b/Lib/concurrent/futures/_base.py --- a/Lib/concurrent/futures/_base.py +++ b/Lib/concurrent/futures/_base.py @@ -49,7 +49,7 @@ class CancelledError(Error): """The Future was cancelled.""" pass -class TimeoutError(Error): +class TimeoutError(Error, TimeoutError): """The operation exceeded the given deadline.""" pass |
|||
| msg217402 - (view) | Author: Guido van Rossum (gvanrossum) * (Python committer) | Date: 2014年04月28日 19:26 | |
I considered this, and decided against unifying the two TimeoutErrors. First the builtin TimeoutError is specifically a subclass of OSError representing the case where errno is ETIMEDOUT. But asyncio.TimeoutError means nothing of the sort. Second, the precedent is concurrent.futures.TimeoutError. The asyncio one is used under the same conditions as that one. I think we should just update the links in the docs to be correct. |
|||
| msg219310 - (view) | Author: Roundup Robot (python-dev) (Python triager) | Date: 2014年05月28日 22:05 | |
New changeset 6d90e8df01f4 by Victor Stinner in branch '3.4': Issue #21376: document asyncio.TimeoutError http://hg.python.org/cpython/rev/6d90e8df01f4 New changeset 03bb1077b362 by Victor Stinner in branch 'default': (Merge 3.4) Issue #21376: document asyncio.TimeoutError http://hg.python.org/cpython/rev/03bb1077b362 |
|||
| msg219311 - (view) | Author: STINNER Victor (vstinner) * (Python committer) | Date: 2014年05月28日 22:06 | |
Thanks for the report. In fact, asyncio.TimeoutError was not documented at all. It should now be fixed. |
|||
| History | |||
|---|---|---|---|
| Date | User | Action | Args |
| 2022年04月11日 14:58:02 | admin | set | github: 65575 |
| 2014年05月28日 22:06:16 | vstinner | set | status: open -> closed resolution: fixed messages: + msg219311 |
| 2014年05月28日 22:05:25 | python-dev | set | nosy:
+ python-dev messages: + msg219310 |
| 2014年04月28日 19:26:12 | gvanrossum | set | nosy:
gvanrossum, pitrou, vstinner, giampaolo.rodola, docs@python, yselivanov, qmega messages: + msg217402 |
| 2014年04月28日 19:20:55 | pitrou | set | nosy:
+ gvanrossum, pitrou, vstinner, giampaolo.rodola, yselivanov messages: + msg217400 assignee: docs@python -> components: + Library (Lib) |
| 2014年04月28日 18:05:57 | qmega | create | |