safwank/ElixirRetry

Current jitter implementation is not very practical

a3kov opened this issue · 1 comments

a3kov commented

It is too extreme IMO - if you include it in the stream pipe, it can completely break the base stream. For example, with exponential backoff, it can produce values that don't grow or even go lower. It should probably work in a percentage margin based on the original value, where the percentage is provided as the function argument.

I understand it's very easy to make own implementation, but maybe this will help future users of this library if fixed.

a3kov commented

Actually what I expected from jitter is implemented by randomize