Decorator should fail safe

[johnarnold]

Scenario: Using redis store, and the store is remote

Expected behavior: If the store is unavailable, catch exceptions, log a warning, and just run the normal function.

Actual behavior: exceptions bubble up

[johnarnold]

An example workaround, credit @tombo315

def memo_safe(**opts):
    """
    Memoizes a function (fail-safe)
    :param opts: options to pass to Memoize call
    :return: the wrapped function.
    """

    def outer(f):
        @wraps(f)
        def inner(*args, **kwargs):

            if memo is None:
                return f(*args, **kwargs)

            try:
                m_func = memo(f, **opts)
                ret = m_func(*args, **kwargs)
            except Exception as e:
                logger.warning("On memoized call: {}".format(str(e)))
                ret = f(*args, **kwargs)
            return ret

        return inner

    return outer