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

[EXPORTER] Support handling retry-able errors for OTLP/HTTP #3223

Merged
merged 35 commits into from
Jan 17, 2025
Merged
Changes from 1 commit
Commits
Show all changes
35 commits
Select commit Hold shift + click to select a range
c4c31aa
Add environment-related logic
chusitoo Dec 24, 2024
ed07242
Update options with retry policy settings
chusitoo Dec 24, 2024
1dba246
Test config and retry mechanism
chusitoo Dec 24, 2024
ae47b20
Functional retry policy
chusitoo Dec 29, 2024
3c2c24a
Model exponential backoff after grpc retry policy
chusitoo Dec 29, 2024
2f9b629
Fix some neglected bits
chusitoo Dec 30, 2024
9cd4717
Reduce usage of decimal seconds to http client et al
chusitoo Dec 30, 2024
cec3b98
Fix overlook on iterator increment
chusitoo Dec 30, 2024
082597a
Unit test http retries with exponential backoff
chusitoo Dec 30, 2024
8008e8c
Update comments for clarity
chusitoo Dec 30, 2024
48402d9
Fix unit tests and minor touchups
chusitoo Dec 30, 2024
6de20c1
Make intent clear in options used by UTs
chusitoo Dec 31, 2024
4de2f81
Cleanup
chusitoo Dec 31, 2024
413b284
Fix to support windows macro
chusitoo Jan 4, 2025
14a2927
Merge branch 'main' into RetryableErrorHttp
marcalff Jan 6, 2025
25b332c
Code review feedback
chusitoo Jan 7, 2025
f511021
Code review feedback pt 2
chusitoo Jan 8, 2025
20b347f
Code review feedback pt 3
chusitoo Jan 8, 2025
da7c636
Change to strtoull to ensure portability on msvc
chusitoo Jan 8, 2025
cd816b6
Include errno
chusitoo Jan 8, 2025
169b4a6
Bail out earlier from retry list
chusitoo Jan 8, 2025
62e2a6a
Merge remote-tracking branch 'remotes/origin/main' into RetryableErro…
chusitoo Jan 8, 2025
2e3ec77
Fixes for iwyu
chusitoo Jan 9, 2025
cb60f09
Fixes for clang-tidy
chusitoo Jan 9, 2025
f8e10c7
Add to changelog
chusitoo Jan 9, 2025
0e33217
Make it an opt-in feature
chusitoo Jan 10, 2025
e8bdb66
Guard includes for iwyu
chusitoo Jan 10, 2025
904929a
Merge remote-tracking branch 'remotes/origin/main' into RetryableErro…
chusitoo Jan 11, 2025
f8b6a09
Slightly faster code, slightly clearer intent
chusitoo Jan 11, 2025
797fda1
Remove unused include
chusitoo Jan 11, 2025
472aa35
Merge remote-tracking branch 'remotes/origin/main' into RetryableErro…
chusitoo Jan 16, 2025
facc19d
Prevent background thread from exiting when pending retries
chusitoo Jan 17, 2025
d3c52fe
Merge remote-tracking branch 'remotes/origin/main' into RetryableErro…
chusitoo Jan 17, 2025
63d7d7d
Address nit and minor logic flaw
chusitoo Jan 17, 2025
905e7ae
Fix clang tidy warning
chusitoo Jan 17, 2025
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Merge remote-tracking branch 'remotes/origin/main' into RetryableErro…
…rHttp
  • Loading branch information
chusitoo committed Jan 8, 2025
commit 62e2a6acf88903e2fa72f91b42b22c9e96366dfd
2 changes: 1 addition & 1 deletion ext/src/http/client/curl/http_operation_curl.cc
Original file line number Diff line number Diff line change
@@ -9,13 +9,13 @@
#include <atomic>
#include <chrono>
#include <cmath>
#include <cstdint>
#include <cstring>
#include <functional>
#include <future>
#include <map>
#include <memory>
#include <random>
#include <sstream>
#include <string>
#include <thread>
#include <utility>
Loading
You are viewing a condensed version of this merge commit. You can view the full changes here.