-
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
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.86% +0.03%
==========================================
Files 368 368
Lines 57975 58008 +33
==========================================
+ Hits 41641 41685 +44
+ Misses 16334 16323 -11
🚀 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?
|
/merge |
|
View all feedbacks in Devflow UI.
The expected merge time in
|
What does this PR do?
This PR modifies
send_datamodule andsend_with_retryfunction so they can accept anHttpClient. This leads to some benefits: