Browse Source

Use the Future's result directly rather than awaiting again

The asyncio documentation does not specify whether awaiting a Future multiple times is supported or not: https://bugs.python.org/issue41275
master
JustAnotherArchivist 3 years ago
parent
commit
749158b97a
1 changed files with 1 additions and 2 deletions
  1. +1
    -2
      qwarc/__init__.py

+ 1
- 2
qwarc/__init__.py View File

@@ -248,9 +248,8 @@ class QWARC:
self._sleepTasks.remove(future)
elif future.taskType == 'process':
item = future.item
# TODO Replace all of this with `if future.cancelled():`
try:
await future #TODO: Is this actually necessary? asyncio.wait only returns 'done' futures...
future.result()
except concurrent.futures.CancelledError as e:
# Got cancelled, nothing we can do about it, but let's log a warning if it's a process task
if future.taskType == 'process':


Loading…
Cancel
Save