-
Notifications
You must be signed in to change notification settings - Fork 15
Reuse connections #1249
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
base: main
Are you sure you want to change the base?
Reuse connections #1249
Conversation
ced7dc8
to
5b91692
Compare
BenchmarksComparisonBenchmark execution time: 2025-10-06 08:48:37 Comparing candidate commit 386bc6b in PR branch Found 0 performance improvements and 0 performance regressions! Performance is the same for 53 metrics, 2 unstable metrics. CandidateCandidate benchmark detailsGroup 1
Group 2
Group 3
Group 4
Group 5
Group 6
Group 7
Group 8
Group 9
Group 10
Group 11
Group 12
Group 13
Group 14
Group 15
BaselineOmitted due to size. |
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## main #1249 +/- ##
==========================================
+ Coverage 71.82% 71.85% +0.03%
==========================================
Files 356 356
Lines 56685 56698 +13
==========================================
+ Hits 40716 40743 +27
+ Misses 15969 15955 -14
🚀 New features to boost your workflow:
|
555352d
to
1805508
Compare
Artifact Size Benchmark Reportaarch64-alpine-linux-musl
aarch64-unknown-linux-gnu
libdatadog-x64-windows
libdatadog-x86-windows
x86_64-alpine-linux-musl
x86_64-unknown-linux-gnu
|
…he agent is reused.
58e37dd
to
386bc6b
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Has this been cleared with Serverless?
What does this PR do?
This PR modifies
send_data
module andsend_with_retry
function so they can accept anHttpClient
. This leads to some benefits: