New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Removing new random source due to high memory cost. #58

Merged

cenkalti merged 1 commit into cenkalti:master from hpidcock:master

Sep 21, 2018

Contributor

hpidcock commented Jun 8, 2018

Instead relying on the default random source.
Random seeding should be handled by the application.

~10k instances of ExponentialBackOff consuming 70MB


          Removing new random source due to high memory cost.

40a8ae2

Instead relying on the default random source.

coveralls commented Jun 8, 2018 •

edited

Coverage increased (+0.8%) to 84.181% when pulling 40a8ae2 on hpidcock:master into f756bc9 on cenkalti:master.

cenkalti merged commit adb73d5 into cenkalti:master

cenkalti mentioned this pull request

Performance issue with exponential backoff #63

Closed

cenkalti mentioned this pull request

fix: exponential random value not change #91

Closed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment