Add a pause when only retries are in the queue

[ianco]

Signed-off-by: Ian Costanzo ian@anon-solutions.ca

Added a sleep() when there are only retries in the queue.

Was causing problems with aries-vcr on startup - when the request was sent to establish a self-connection, aca-py was hogging cpu and preventing the aries-vcr startup from continuing; aries-vcr was unable to respond to callbacks until aca-py retries had all failed and timed out.

[codecov-commenter]

Codecov Report

Merging #636 into master will decrease coverage by 0.01%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master     #636      +/-   ##
==========================================
- Coverage   97.51%   97.50%   -0.02%     
==========================================
  Files         250      250              
  Lines       13345    13347       +2     
==========================================
  Hits        13014    13014              
- Misses        331      333       +2     

[nrempel]

Good catch. await asyncio.sleep(0) should (I think?) have essentially the same effect and avoid the use of magic numbers. But perhaps we want a slightly longer delay in any case.

[ianco]

Good catch. await asyncio.sleep(0) should (I think?) have essentially the same effect and avoid the use of magic numbers. But perhaps we want a slightly longer delay in any case.

I think sleep(0.05) is ok, a short delay doesn't hurt (the retry interval is much longer anyways), and the magic number is used in other delays elsewhere in aca-py