Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix recursion while waiting for redis connection #311

Merged
merged 3 commits into from
Aug 23, 2022

Commits on May 19, 2022

  1. Fix recursion while waiting for redis connection

    `Arq` has ability to retry connections to redis in case of problems.
    That is a nice feature, but it crashes after hundreds of attempts
    reaching maximal recursion depth. This change modifies re-try algorithm
    from recursion to iteration avoiding the limit.
    
    In practice it would be nice to have ability to wait forever as issue
    with redis instance should not kill worker process, but that is a
    separate change that can be built on top.
    
    Fixes exception when retrying redis connection many times:
      RecursionError: maximum recursion depth exceeded while getting the str of an object
    nierob committed May 19, 2022
    Configuration menu
    Copy the full SHA
    f248aed View commit details
    Browse the repository at this point in the history

Commits on Aug 23, 2022

  1. Configuration menu
    Copy the full SHA
    3ff25ad View commit details
    Browse the repository at this point in the history
  2. speedup test

    samuelcolvin committed Aug 23, 2022
    Configuration menu
    Copy the full SHA
    2eab750 View commit details
    Browse the repository at this point in the history