search

nodeshift / opossum

Node.js circuit breaker - fails fast ⚡️
https://nodeshift.dev/opossum/
Apache License 2.0
1.33k stars 107 forks source link

Lack of Count-Based Rolling Window #817

Open slice-arpitkhatri opened 1 year ago

slice-arpitkhatri commented 1 year ago

As per my understanding, we currently do not support a count-based rolling window. I want CB to act based on the last 50-odd requests. Since my TPS fluctuates between day and night, I have to adjust my rollingCountWindow based on TPS to ensure that there are at least 50 requests in the specific window.

Would it make sense to add count-based rolling window support?

lholmquist commented 1 year ago

Could be interesting. If this is something you would like to work on, send a PR and we could walk through it

github-actions[bot] commented 1 year ago

This issue is stale because it has been open 30 days with no activity.

github-actions[bot] commented 7 months ago

This issue is stale because it has been open 30 days with no activity.