diff options
author | Miss Islington (bot) <31488909+miss-islington@users.noreply.github.com> | 2020-05-31 07:26:20 (GMT) |
---|---|---|
committer | GitHub <noreply@github.com> | 2020-05-31 07:26:20 (GMT) |
commit | cdb015b7ed58ee2babf552001374c278219854e1 (patch) | |
tree | f5e66d38ece32f20f46633355f0a4d0aa9b6f72f /Lib/asyncio | |
parent | 44400e88f906bf99cd2c57791f1201c400daf6af (diff) | |
download | cpython-cdb015b7ed58ee2babf552001374c278219854e1.zip cpython-cdb015b7ed58ee2babf552001374c278219854e1.tar.gz cpython-cdb015b7ed58ee2babf552001374c278219854e1.tar.bz2 |
Fix asyncio.to_thread() documented return type (GH-20547)
When I wrote the documentation for `asyncio.to_thread()`, I mistakenly assumed that `return await loop.run_in_executor(...)` within an async def function would return a Future. In reality, it returns a coroutine.
This likely won't affect typical usage of `asyncio.to_thread()`, but it's important for the documentation to be correct here. In general, we also tend to avoid returning futures from high-level APIs in asyncio.
(cherry picked from commit 2b201369b435a4266bda5b895e3b615dbe28ea6e)
Co-authored-by: Kyle Stanley <aeros167@gmail.com>
Diffstat (limited to 'Lib/asyncio')
-rw-r--r-- | Lib/asyncio/threads.py | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/Lib/asyncio/threads.py b/Lib/asyncio/threads.py index 51e0ba9..34b7513 100644 --- a/Lib/asyncio/threads.py +++ b/Lib/asyncio/threads.py @@ -17,7 +17,7 @@ async def to_thread(func, /, *args, **kwargs): allowing context variables from the main thread to be accessed in the separate thread. - Return an asyncio.Future which represents the eventual result of *func*. + Return a coroutine that can be awaited to get the eventual result of *func*. """ loop = events.get_running_loop() ctx = contextvars.copy_context() |