Skip to content

fix: If the brpoplpush commands fails, wait before retrying#545

Closed
compwright wants to merge 4 commits intobee-queue:masterfrom
compwright:backoff-command-retries
Closed

fix: If the brpoplpush commands fails, wait before retrying#545
compwright wants to merge 4 commits intobee-queue:masterfrom
compwright:backoff-command-retries

Conversation

@compwright
Copy link
Copy Markdown
Collaborator

Copy of #162 by @tmeasday, without checks for Node < v12:

The current behaviour is to retry as fast as possible, leading to a spam of commands and error messages if the redis instance is for example unavailable for a brief period. It seems more sensible to take a bit more care in retrying.

@compwright compwright force-pushed the backoff-command-retries branch from 3ac6bbd to 415ddcd Compare November 22, 2022 16:28
@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage remained the same at 100.0% when pulling 415ddcd on compwright:backoff-command-retries into 41710a8 on bee-queue:master.

@compwright compwright closed this Nov 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants