ixti / sidekiq-throttled

Concurrency and rate-limit throttling for Sidekiq
MIT License
698 stars 75 forks source link

refactor!: Inherit Fetch from BasicFetch #147

Closed ixti closed 1 year ago

ixti commented 1 year ago

Remove expirable list

After lots of thoughts the implementation was causing more troubles than benefits. Without thislist we will be thrashing Redis a lot under certain circumstances, but that should be fixed in-place where it makes sense by patching fetcher. We will provide lifecycle hooks to subscribe that will allow patch fetcher's queue list implementation.