Skip to content
This repository has been archived by the owner on Mar 21, 2019. It is now read-only.

Add jitter #19

Open
lvh opened this issue Apr 25, 2017 · 0 comments
Open

Add jitter #19

lvh opened this issue Apr 25, 2017 · 0 comments

Comments

@lvh
Copy link
Contributor

lvh commented Apr 25, 2017

Right now, we have deterministic exponential backoff. This means users of banach potentially are exposed to stampeding herd problems.

We could do this in exponentially, or make it composable.

A composable API would look like [jitter-params strategy] -> strategy. If we use with-clock with some state, we can figure out what delays another strategy imposed and instead apply jitter and then run them with a real clock (unless if course it's composable turtles all the way down, life is a simulation, and all attempts at computer are intrinsically futile). This is not super great, but we didn't make up that global mutable state that is the reactoclock.

The benefit of a composable API is that it's reusable. The default, especially for helper fns, should absolutely include jitter.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant