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.

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

feat: add exponential backoff for sending data #76

Merged
merged 11 commits into from
Apr 22, 2022

Conversation

sumo-drosiek
Copy link
Contributor

@sumo-drosiek sumo-drosiek commented Apr 21, 2022

This PR adds exponential backoff while sending data due to the following issue

in current state if one of request in chunk fails, full chunk is going to be retried which causes data duplication and increases ingest (and costs) for customers

@sumo-drosiek sumo-drosiek force-pushed the drosiek-retry-forever branch 2 times, most recently from a1beda6 to 4bae178 Compare April 21, 2022 11:06
@sumo-drosiek sumo-drosiek force-pushed the drosiek-retry-forever branch from 4bae178 to 3542ba0 Compare April 21, 2022 11:07
@sumo-drosiek sumo-drosiek force-pushed the drosiek-retry-forever branch from 5ed48fb to 64c83ed Compare April 21, 2022 11:59
@sumo-drosiek sumo-drosiek marked this pull request as ready for review April 21, 2022 12:03
@sumo-drosiek sumo-drosiek requested a review from a team as a code owner April 21, 2022 12:03
Signed-off-by: Dominik Rosiek <[email protected]>
Dominik Rosiek added 2 commits April 21, 2022 14:52
Signed-off-by: Dominik Rosiek <[email protected]>
sumo-drosiek and others added 2 commits April 22, 2022 08:56
Co-authored-by: Andrzej Stencel <[email protected]>
Co-authored-by: Andrzej Stencel <[email protected]>
@sumo-drosiek sumo-drosiek merged commit 417b123 into main Apr 22, 2022
@sumo-drosiek sumo-drosiek deleted the drosiek-retry-forever branch April 22, 2022 07:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants