-
Notifications
You must be signed in to change notification settings - Fork 15
feat(profiling): File endpoint for exporter #1421
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?
Conversation
Artifact Size Benchmark Reportaarch64-alpine-linux-musl
aarch64-apple-darwin
aarch64-unknown-linux-gnu
libdatadog-x64-windows
libdatadog-x86-windows
x86_64-alpine-linux-musl
x86_64-apple-darwin
x86_64-unknown-linux-gnu
|
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## main #1421 +/- ##
==========================================
+ Coverage 71.41% 71.55% +0.13%
==========================================
Files 411 413 +2
Lines 66028 66426 +398
==========================================
+ Hits 47156 47528 +372
- Misses 18872 18898 +26
🚀 New features to boost your workflow:
|
3846d3d to
3e70843
Compare
3e70843 to
2632ccf
Compare
BenchmarksComparisonBenchmark execution time: 2025-12-19 17:21:22 Comparing candidate commit 8c22595 in PR branch Found 0 performance improvements and 0 performance regressions! Performance is the same for 57 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
Group 16
Group 17
Group 18
Group 19
BaselineOmitted due to size. |
What does this PR do?
Adds a
file://endpoint to the profile exporter.Motivation
For testing and debugging purposes, its really useful to see exactly what bytes were sent. This is useful both within libdatadog, and also for any clients.
Additional Notes
Anything else we should know when reviewing?
How to test the change?
New integration tests in the PR.