-
Notifications
You must be signed in to change notification settings - Fork 15
profiling: use C++ api #1382
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
profiling: use C++ api #1382
Conversation
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## main #1382 +/- ##
==========================================
+ Coverage 71.22% 71.48% +0.26%
==========================================
Files 404 405 +1
Lines 64477 65032 +555
==========================================
+ Hits 45923 46491 +568
+ Misses 18554 18541 -13
🚀 New features to boost your workflow:
|
BenchmarksComparisonBenchmark execution time: 2025-12-15 22:50:47 Comparing candidate commit 74880e9 in PR branch Found 12 performance improvements and 1 performance regressions! Performance is the same for 44 metrics, 2 unstable metrics. scenario:credit_card/is_card_number/ 378282246310005
scenario:credit_card/is_card_number/378282246310005
scenario:credit_card/is_card_number/37828224631000521389798
scenario:credit_card/is_card_number_no_luhn/ 378282246310005
scenario:credit_card/is_card_number_no_luhn/378282246310005
scenario:credit_card/is_card_number_no_luhn/37828224631000521389798
scenario:sql/obfuscate_sql_string
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. |
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
|
01042af to
f3d75a3
Compare
6b243aa to
6398405
Compare
6398405 to
74880e9
Compare
|
/merge |
|
View all feedbacks in Devflow UI.
The expected merge time in
|
What does this PR do?
Provides a C++ api for profiling
Motivation
A type-safe ideomatic interface. What's not to love?
Additional Notes
Anything else we should know when reviewing?
How to test the change?
Run the new CPP example.