Skip to content
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

sync -- add exponential backoff #1684

Merged
merged 2 commits into from
Jul 3, 2023
Merged

sync -- add exponential backoff #1684

merged 2 commits into from
Jul 3, 2023

Conversation

danlaine
Copy link

@danlaine danlaine commented Jul 3, 2023

Why this should be merged

Prevent retry storms.

Table of attempt number and timeout:

1 10ms
2 20ms
3 30ms
4 50ms
5 70ms
6 110ms
7 170ms
8 250ms
9 380ms
10 570ms
11 860ms
12 1s

all subsequent requests will wait 1s

How this works

Add exponential backoff

How this was tested

Existing UT

@danlaine danlaine self-assigned this Jul 3, 2023
@danlaine danlaine merged commit 9b3b955 into sync-refactor-client Jul 3, 2023
11 checks passed
@danlaine danlaine deleted the sync-add-exponential-backoff branch July 3, 2023 19:20
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@darioush darioush Awaiting requested review from darioush

@dboehm-avalabs dboehm-avalabs Awaiting requested review from dboehm-avalabs

Assignees

@danlaine danlaine

Labels
merkledb
Projects
No open projects
Ski Patrol
Archived in project
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@danlaine