diff options
author | cyrus durgin <cyrusd@gmail.com> | 2015-09-10 15:57:35 -0700 |
---|---|---|
committer | cyrus durgin <cyrusd@gmail.com> | 2015-09-10 15:57:35 -0700 |
commit | 6a55573d588e4a817986c8964d97dceb8c46585a (patch) | |
tree | c578059a5b8ac5d3adc7a8e2c92dfe82ca27b2ff /retrying.py | |
parent | da33537e0850f997aea75c4dec010e2b5594071f (diff) | |
download | retrying-6a55573d588e4a817986c8964d97dceb8c46585a.tar.gz |
Adds two additional parameters to the Retrying() constructor:
_before_attempts
_after_attempts
Each of these should be a reference to a function that takes a
single argument, attempt_number.
_before_attempts will be called before each attempt is run;
_after_attempts is run after any *unsuccessful* attempt.
Handy for logging status messages, etc in between attempts.
Diffstat (limited to 'retrying.py')
-rw-r--r-- | retrying.py | 12 |
1 files changed, 11 insertions, 1 deletions
diff --git a/retrying.py b/retrying.py index 3ed312d..79111ae 100644 --- a/retrying.py +++ b/retrying.py @@ -68,7 +68,9 @@ class Retrying(object): wrap_exception=False, stop_func=None, wait_func=None, - wait_jitter_max=None): + wait_jitter_max=None, + before_attempts=None, + after_attempts=None): self._stop_max_attempt_number = 5 if stop_max_attempt_number is None else stop_max_attempt_number self._stop_max_delay = 100 if stop_max_delay is None else stop_max_delay @@ -80,6 +82,8 @@ class Retrying(object): self._wait_exponential_multiplier = 1 if wait_exponential_multiplier is None else wait_exponential_multiplier self._wait_exponential_max = MAX_WAIT if wait_exponential_max is None else wait_exponential_max self._wait_jitter_max = 0 if wait_jitter_max is None else wait_jitter_max + self._before_attempts = before_attempts + self._after_attempts = after_attempts # TODO add chaining of stop behaviors # stop behavior @@ -196,6 +200,9 @@ class Retrying(object): start_time = int(round(time.time() * 1000)) attempt_number = 1 while True: + if self._before_attempts: + self._before_attempts(attempt_number) + try: attempt = Attempt(fn(*args, **kwargs), attempt_number, False) except: @@ -205,6 +212,9 @@ class Retrying(object): if not self.should_reject(attempt): return attempt.get(self._wrap_exception) + if self._after_attempts: + self._after_attempts(attempt_number) + delay_since_first_attempt_ms = int(round(time.time() * 1000)) - start_time if self.stop(attempt_number, delay_since_first_attempt_ms): if not self._wrap_exception and attempt.has_exception: |