> @@ -126,6 +127,9 @@ public void imposeBackoffExponentialDelay(long period,
> int pow, int failureCount
> public void imposeBackoffExponentialDelay(long period, long maxPeriod,
> int pow, int failureCount, int max,
> String commandDescription) {
> long delayMs = (long) (period * Math.pow(failureCount, pow));
> + // Add random delay to avoid thundering herd problem when multiple
> + // simultaneous failed requests retry after sleeping for the same
> delay.
> + delayMs += new Random().nextInt((int) (delayMs / 10));
> since we only see this on the uncommon failure path.
Ah, OK. Then let's keep as-is. Just wondering if we want to change the jitter
to min(5ms, delay * 0.1) or so?
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/286/files#r9649037
