pnpnpn / retry-decorator

Decorator for retrying when exceptions occur

Usage

Retry decorator

#!/usr/bin/env python

from __future__ import print_function
from retry_decorator import *

@retry(Exception, tries = 3, timeout_secs = 0.1)
def test_retry():
    import sys
    print('hello', file = sys.stderr)
    raise Exception('Testing retry')

if __name__ == '__main__':
    try:
        test_retry()
    except Exception as e:
        print('Received the last exception')

Contribute

I would love for you to fork and send me pull request for this project. Please contribute.

retry-decorator's Issues

why is run_one_last_time set to False if callback_by_exception is defined

See https://github.com/pnpnpn/retry-decorator/blob/master/retry_decorator/retry_decorator.py#L34:
when callback_by_exception is defined and in effect, then run_one_last_time is defaulted to False.

1. why is it so?
2. what's the purpose of run_one_last_time to begin with? To be able to swallow exceptions by the retry logic?

In any case, run_one_last_time should always default to True.

Please add per-file licenses

The Chromium project (www.chromium.org) pulls in retry-decorator indirectly through the catapult (https://github.com/catapult-project/catapult/) and gsutil (https://github.com/GoogleCloudPlatform/gsutil) source repositories. In order for Chromium to be pulled into various Linux source distributions there's a requirement that all of the third party files pass the Linux licensecheck utility. Currently there are many files in the retry-decorator repository missing per-file licenses. From a current run of licensecheck, they are:

$ licensecheck -r . | grep "No copyright"
./setup.py: *No copyright* UNKNOWN
./retry_decorator/__init__.py: *No copyright* UNKNOWN
./retry_decorator/retry_decorator.py: *No copyright* UNKNOWN

We'd like to ask that per-file licenses be added to these files to make it easier to integrate retry-decorator not only into Chromium, but also Linux distributions in general. Thanks.

universal wheel

Hi!

Could you add a setup.cfg and enable the bdist universal option?

[bdist_wheel]
universal = 1

It would make installing the library via py2 and py3 easier.

And then upload this wheel to PyPI as a new version?

execute some code before retrying

Is it possible to execute some code before retrying?

For example:

try:		
	return function(base_db, sql_string, sql_vars, debug_sql)
except (AttributeError, MySQLdb.OperationalError):
	base_db.__init__()
	return self.retry_db_errors(function(base_db, sql_string, sql_vars, debug_sql))

git tag for 1.1.0

Could you please tag 1.1.0?

timeout_secs is confusing

that parameter name should be replaced with "wait_secs" or "interval_secs" or something - it doesn't look like it's a timeout at all, but the interval between retries from what I can see.

Multiple Exception

Is it possible to use this extension for multiple exceptions?

CHANGES.txt and tests/ in pypi tarball

Could you please consider to add tests/ and CHANGES.txt into release?