-
Notifications
You must be signed in to change notification settings - Fork 500
[EXPORTER] Support handling retry-able errors for OTLP/HTTP #3223
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
Merged
Merged
Changes from all commits
Commits
Show all changes
35 commits
Select commit
Hold shift + click to select a range
c4c31aa
Add environment-related logic
chusitoo ed07242
Update options with retry policy settings
chusitoo 1dba246
Test config and retry mechanism
chusitoo ae47b20
Functional retry policy
chusitoo 3c2c24a
Model exponential backoff after grpc retry policy
chusitoo 2f9b629
Fix some neglected bits
chusitoo 9cd4717
Reduce usage of decimal seconds to http client et al
chusitoo cec3b98
Fix overlook on iterator increment
chusitoo 082597a
Unit test http retries with exponential backoff
chusitoo 8008e8c
Update comments for clarity
chusitoo 48402d9
Fix unit tests and minor touchups
chusitoo 6de20c1
Make intent clear in options used by UTs
chusitoo 4de2f81
Cleanup
chusitoo 413b284
Fix to support windows macro
chusitoo 14a2927
Merge branch 'main' into RetryableErrorHttp
marcalff 25b332c
Code review feedback
chusitoo f511021
Code review feedback pt 2
chusitoo 20b347f
Code review feedback pt 3
chusitoo da7c636
Change to strtoull to ensure portability on msvc
chusitoo cd816b6
Include errno
chusitoo 169b4a6
Bail out earlier from retry list
chusitoo 62e2a6a
Merge remote-tracking branch 'remotes/origin/main' into RetryableErro…
chusitoo 2e3ec77
Fixes for iwyu
chusitoo cb60f09
Fixes for clang-tidy
chusitoo f8e10c7
Add to changelog
chusitoo 0e33217
Make it an opt-in feature
chusitoo e8bdb66
Guard includes for iwyu
chusitoo 904929a
Merge remote-tracking branch 'remotes/origin/main' into RetryableErro…
chusitoo f8b6a09
Slightly faster code, slightly clearer intent
chusitoo 797fda1
Remove unused include
chusitoo 472aa35
Merge remote-tracking branch 'remotes/origin/main' into RetryableErro…
chusitoo facc19d
Prevent background thread from exiting when pending retries
chusitoo d3c52fe
Merge remote-tracking branch 'remotes/origin/main' into RetryableErro…
chusitoo 63d7d7d
Address nit and minor logic flaw
chusitoo 905e7ae
Fix clang tidy warning
chusitoo File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.