Skip to content

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

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

Auto-infer instrumentation required helper classes at build time#12059
sarahchen6 wants to merge 8 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

This comment has been minimized.

@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 — All performance SLOs passed

Suite Status
Startup 🟢 pass

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 14.03 s 13.94 s [-0.1%; +1.4%] (no difference)
startup:insecure-bank:tracing:Agent 12.97 s 13.02 s [-1.0%; +0.2%] (no difference)
startup:petclinic:appsec:Agent 17.39 s 17.26 s [-0.3%; +1.8%] (no difference)
startup:petclinic:iast:Agent 16.70 s 17.53 s [-8.9%; -0.6%] (maybe better)
startup:petclinic:profiling:Agent 17.49 s 17.29 s [-0.1%; +2.3%] (no difference)
startup:petclinic:sca:Agent 17.50 s 17.48 s [-1.1%; +1.2%] (no difference)
startup:petclinic:tracing:Agent 16.53 s 16.65 s [-1.8%; +0.4%] (no difference)

Commit: fde64f41 · 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