Skip to content

Add backoff for failed dequeue operations - #10

Open
samuel-williams-shopify wants to merge 1 commit into
mainfrom
fix/dequeue-backoff
Open

Add backoff for failed dequeue operations#10
samuel-williams-shopify wants to merge 1 commit into
mainfrom
fix/dequeue-backoff

Conversation

@samuel-williams-shopify

Copy link
Copy Markdown
Contributor

Summary

  • retry failed dequeue operations using configurable exponential backoff with jitter
  • cap retry delays and reset them after a successful dequeue
  • treat an empty result from the indefinitely blocking Redis dequeue as an I/O error
  • cover retry behavior, delay reset, and empty dequeue results

Verification

  • bundle exec bake test
  • bundle exec rubocop
  • bundle exec bake decode:index:coverage lib
  • 100% line coverage from a fresh coverage database

Fixes #6

Assisted-By: devx/dd27ce98-f6f5-4716-8b1d-39238c972003
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.

Server#start! dequeue loop spins CPU at 100% when Redis connection fails

1 participant