projects
/
python_utils.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
tree
raw
|
inline
| side by side
Cleanup unittest_utils trying to get clean lint.
[python_utils.git]
/
smart_future.py
diff --git
a/smart_future.py
b/smart_future.py
index 1f6e6f0aedcf05966e536ec8f10f570c2175a3e4..9aa68f38a5396953636e257032542f9d07533225 100644
(file)
--- a/
smart_future.py
+++ b/
smart_future.py
@@
-1,18
+1,24
@@
#!/usr/bin/env python3
#!/usr/bin/env python3
+"""A future that can be treated like the result that it contains and
+will not block until it is used. At that point, if the underlying
+value is not yet available, it will block until it becomes
+available."""
+
from __future__ import annotations
from __future__ import annotations
+
import concurrent
import concurrent.futures as fut
import logging
import concurrent
import concurrent.futures as fut
import logging
-import traceback
from typing import Callable, List, Set, TypeVar
from overrides import overrides
from typing import Callable, List, Set, TypeVar
from overrides import overrides
+import id_generator
+
# This module is commonly used by others in here and should avoid
# taking any unnecessary dependencies back on them.
from deferred_operand import DeferredOperand
# This module is commonly used by others in here and should avoid
# taking any unnecessary dependencies back on them.
from deferred_operand import DeferredOperand
-import id_generator
logger = logging.getLogger(__name__)
logger = logging.getLogger(__name__)
@@
-29,7
+35,7
@@
def wait_any(
smart_future_by_real_future = {}
completed_futures: Set[fut.Future] = set()
for x in futures:
smart_future_by_real_future = {}
completed_futures: Set[fut.Future] = set()
for x in futures:
- assert
type(x) == SmartFuture
+ assert
isinstance(x, SmartFuture)
real_futures.append(x.wrapped_future)
smart_future_by_real_future[x.wrapped_future] = x
real_futures.append(x.wrapped_future)
smart_future_by_real_future[x.wrapped_future] = x
@@
-42,15
+48,12
@@
def wait_any(
if log_exceptions and not f.cancelled():
exception = f.exception()
if exception is not None:
if log_exceptions and not f.cancelled():
exception = f.exception()
if exception is not None:
- logger.warning(
- f'Future {id(f)} raised an unhandled exception and exited.'
- )
+ logger.warning('Future 0x%x raised an unhandled exception and exited.', id(f))
logger.exception(exception)
raise exception
yield smart_future_by_real_future[f]
if callback is not None:
callback()
logger.exception(exception)
raise exception
yield smart_future_by_real_future[f]
if callback is not None:
callback()
- return
def wait_all(
def wait_all(
@@
-60,7
+63,7
@@
def wait_all(
) -> None:
real_futures = []
for x in futures:
) -> None:
real_futures = []
for x in futures:
- assert
type(x) == SmartFuture
+ assert
isinstance(x, SmartFuture)
real_futures.append(x.wrapped_future)
(done, not_done) = concurrent.futures.wait(
real_futures.append(x.wrapped_future)
(done, not_done) = concurrent.futures.wait(
@@
-71,9
+74,7
@@
def wait_all(
if not f.cancelled():
exception = f.exception()
if exception is not None:
if not f.cancelled():
exception = f.exception()
if exception is not None:
- logger.warning(
- f'Future {id(f)} raised an unhandled exception and exited.'
- )
+ logger.warning('Future 0x%x raised an unhandled exception and exited.', id(f))
logger.exception(exception)
raise exception
assert len(done) == len(real_futures)
logger.exception(exception)
raise exception
assert len(done) == len(real_futures)
@@
-89,7
+90,7
@@
class SmartFuture(DeferredOperand):
"""
def __init__(self, wrapped_future: fut.Future) -> None:
"""
def __init__(self, wrapped_future: fut.Future) -> None:
- assert
type(wrapped_future) == fut.Future
+ assert
isinstance(wrapped_future, fut.Future)
self.wrapped_future = wrapped_future
self.id = id_generator.get("smart_future_id")
self.wrapped_future = wrapped_future
self.id = id_generator.get("smart_future_id")
@@
-102,5
+103,5
@@
class SmartFuture(DeferredOperand):
# You shouldn't have to call this; instead, have a look at defining a
# method on DeferredOperand base class.
@overrides
# You shouldn't have to call this; instead, have a look at defining a
# method on DeferredOperand base class.
@overrides
- def _resolve(self,
*,
timeout=None) -> T:
+ def _resolve(self, timeout=None) -> T:
return self.wrapped_future.result(timeout)
return self.wrapped_future.result(timeout)