Skip to content

Auto-infer instrumentation required helper classes at build time - #12059

Draft
sarahchen6 wants to merge 10 commits into
masterfrom
sarahchen6/infer-helper-classes-pt-1
Draft

Auto-infer instrumentation required helper classes at build time#12059
sarahchen6 wants to merge 10 commits into
masterfrom
sarahchen6/infer-helper-classes-pt-1

Conversation

@sarahchen6

@sarahchen6 sarahchen6 commented Jul 23, 2026

Copy link
Copy Markdown
Contributor

What Does This Do

Implement methodology to auto-infer instrumentations' required helper classes at build time.

Also migrate a few instrumentations from manual declarations of required helper classes to auto-detecting these classes in order to test the new methodology.

Motivation

Currently, instrumentation helper classes are manually listed for each InstrumenterModule. However, we can instead use Muzzle, which already finds class references for each instrumentation, to automatically discover which helper classes need to be loaded per instrumentation. This is similar to how OTel declares helpers.

Additional Notes

The intention is for all instrumentations to discover required helper classes at build time; however, I wanted to keep the scope of this PR low so that it's easier to review. The rest of the instrumentations will be migrated in follow-up PRs.

Contributor Checklist

Jira ticket: [PROJ-IDENT]

@datadog-official

datadog-official Bot commented Jul 23, 2026

Copy link
Copy Markdown
Contributor

🎯 Code Coverage (details)
Patch Coverage: 0.00%
Overall Coverage: 57.49% (-0.09%)

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: 98edf9d | Docs | Datadog PR Page | Give us feedback!

@DataDog DataDog deleted a comment from dd-octo-sts Bot Jul 23, 2026
@dd-octo-sts

dd-octo-sts Bot commented Jul 23, 2026

Copy link
Copy Markdown
Contributor

🔴 Java Benchmark SLOs — Performance SLO breach detected

Suite Status
Startup 🔴 breach

SLO thresholds are defined here based on automatically generated metrics. A warning is raised when results are within 5% of the threshold.

PR vs. master results
Scenario Candidate master Δ (95% CI of mean)
startup:insecure-bank:iast:Agent 15.02 s 14.63 s [+1.9%; +3.4%] (significantly worse)
startup:insecure-bank:tracing:Agent 13.74 s 13.73 s [-0.6%; +0.7%] (no difference)
startup:petclinic:appsec:Agent 17.58 s 17.24 s [+1.0%; +3.0%] (maybe worse)
startup:petclinic:iast:Agent 17.56 s 17.63 s [-1.3%; +0.6%] (no difference)
startup:petclinic:profiling:Agent 17.50 s 17.21 s [+0.5%; +2.9%] (maybe worse)
startup:petclinic:sca:Agent 17.59 s 17.45 s [-0.3%; +1.9%] (no difference)
startup:petclinic:tracing:Agent 16.85 s 16.52 s [+0.8%; +3.1%] (maybe worse)

Commit: 98edf9d5 · CI Pipeline · Benchmarking Platform UI


Load and DaCapo benchmarks can be triggered manually in the GitLab pipeline. Results will appear in the Benchmarking Platform UI after completion.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant