search

jclouds / legacy-jclouds

https://jclouds.apache.org
466 stars 216 forks source link

Add random delay to avoid thundering herd problem #1550

Closed gaul closed 10 years ago

gaul commented 11 years ago

When issuing many simultaneous requests to Synaptic Atmos I observed:

HTTP/1.1 failed with code 500, error: AtmosError [code=1040, message=The server is busy. Please try again.]

Previously all clients slept for fixed intervals and thus retried around the same time. This commit adds a random delay which should better distribute load on the provider.

buildhive commented 11 years ago

Adrian Cole » jclouds #26 UNSTABLE Looks like there's a problem with this pull request (what's this?)

cloudbees-pull-request-builder commented 11 years ago

jclouds-java-7-pull-requests #386 UNSTABLE Looks like there's a problem with this pull request

buildhive commented 11 years ago

Adrian Cole » jclouds #28 UNSTABLE Looks like there's a problem with this pull request (what's this?)

cloudbees-pull-request-builder commented 11 years ago

jclouds-java-7-pull-requests #387 SUCCESS This pull request looks good

gaul commented 11 years ago

BackoffLimitedRetryHandlerTest.testExponentialBackoffDelayDefaultMaxInterval500 fails randomly for me. I need to update some of the asserts.

gaul commented 10 years ago

Superseded by JCLOUDS-460 and jclouds/jclouds#286.

demobox commented 10 years ago

Thanks for cleaning this one up, @andrewgaul!