-
Notifications
You must be signed in to change notification settings - Fork 377
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
DEBUG-2334 remaining dynamic instrumentation code #4098
Conversation
spec/datadog/di/contrib/active_record/serializer_active_record_spec.rb
Outdated
Show resolved
Hide resolved
BenchmarksBenchmark execution time: 2024-11-25 14:40:34 Comparing candidate commit 5fe2b77 in PR branch Found 0 performance improvements and 1 performance regressions! Performance is the same for 30 metrics, 2 unstable metrics. scenario:profiler - sample timeline=false
|
b7e93a0
to
288d1d8
Compare
79e1ff8
to
7d4aedf
Compare
66546cd
to
52034e2
Compare
3f462a9
to
7f94a88
Compare
Maximum capture attribute count can be specified in the probe, implement support for it in DI.
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #4098 +/- ##
========================================
Coverage 97.78% 97.78%
========================================
Files 1350 1353 +3
Lines 81399 81786 +387
Branches 4109 4141 +32
========================================
+ Hits 79593 79974 +381
- Misses 1806 1812 +6 ☔ View full report in Codecov by Sentry. 🚨 Try these New Features:
|
What does this PR do?
is set in environment
Motivation:
Initial DI implementation in Ruby
Change log entry
Dynamic instrumentation is now available in Ruby as a Preview. Currently only log probes are implemented, they can be set on both methods and lines. If you are interested in trying out Dynamic Instrumentation in Ruby, please contact Datadog to be added to the Preview program.
Additional Notes:
Replaces #4063
How to test the change?
Additional unit and integration tests are included + system tests in DataDog/system-tests#3516