Make Java convention the composition root by bric3 · Pull Request #12333 · DataDog/dd-trace-java

bric3 · 2026-08-28T13:54:37Z

What Does This Do

Move dependency locking and the remaining java_deps.gradle and java_no_deps.gradle composition into dd-trace-java.conventions.java.

Delete the now-unreferenced gradle/java.gradle compatibility aggregator and retain the Spotless regression guard with guidance toward the Java convention.

Motivation

After #12331 migrates the remaining direct consumers, the Java convention can become the composition root and own the incremental extraction of its remaining legacy internals.

See #12258.

Contributor Checklist

Format the title according to the contribution guidelines
Assign the type: and (comp: or inst:) labels in addition to any other useful labels
Avoid using close, fix, or any linking keywords when referencing an issue
Use solves instead, and assign the PR milestone to the issue
Update the CODEOWNERS file on source file addition, migration, or deletion
Update public documentation with any new configuration flags or behaviors
Once approved, use merge queue to merge the PR

Keep shared Kotlin and default Groovy test wiring behind plugin callbacks.

Cover JVM test suites and test fixtures through Gradle source-set APIs.

datadog-prod-us1-6

More details

The Java convention keeps the deleted aggregator's plugin and script order. No build script still refers to the deleted file.

Was this helpful? React 👍 or 👎

_{Open Bits AI session}

_{🤖 Datadog Autotest · Commit ebc24bc · What is Autotest? · @DataDog review to ask questions · Any feedback? Reach out in #autotest}

dd-octo-sts · 2026-08-28T14:16:32Z

🟢 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.77 s	14.60 s	[+0.5%; +1.9%] (maybe worse)
startup:insecure-bank:tracing:Agent	13.58 s	13.73 s	[-2.0%; -0.1%] (maybe better)
startup:petclinic:appsec:Agent	17.56 s	17.36 s	[+0.4%; +1.8%] (maybe worse)
startup:petclinic:iast:Agent	16.89 s	17.56 s	[-8.0%; +0.4%] (no difference)
startup:petclinic:profiling:Agent	17.48 s	17.22 s	[+0.4%; +2.6%] (maybe worse)
startup:petclinic:sca:Agent	17.56 s	17.49 s	[-0.5%; +1.3%] (no difference)
startup:petclinic:tracing:Agent	16.73 s	16.76 s	[-1.4%; +1.0%] (no difference)

Commit: 56f0ecfe · 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.

bric3 · 2026-09-04T10:18:49Z

/merge

gh-worker-devflow-routing-ef8351 · 2026-09-04T10:18:53Z

View all feedbacks in Devflow UI.

2026-09-04 10:18:53 UTC ℹ️ Start processing command /merge

2026-09-04 10:19:02 UTC ℹ️ MergeQueue: Pull request is not mergeable yet

It will be processed automatically as soon as GitHub reports it as mergeable. View in MergeQueue UI.

Run /code blockers to see what is blocking it.
Run /remove to cancel it.

2026-09-04 11:20:28 UTC ℹ️ MergeQueue: merge request added to the queue

The expected merge time in master is approximately 1h (p90).

2026-09-04 11:36:33 UTC ❌ MergeQueue: The build pipeline contains failing jobs for this merge request

Build pipeline has failing jobs for cdda3f6:

check_smoke 1/4

⚠️ Do NOT retry failed jobs directly (why?).

What to do next?

Investigate the failures and when ready, re-add your pull request to the queue!
If your PR checks are green, try to rebase/merge. It might be because the CI run is a bit old.
Any question, go check the FAQ.

Details

Since those jobs are not marked as being allowed to fail, the pipeline will most likely fail.
Therefore, and to allow other builds to be processed, this merge request has been rejected and the pipeline got canceled.

bric3 · 2026-09-04T10:31:59Z

/merge

gh-worker-devflow-routing-ef8351 · 2026-09-04T10:32:04Z

View all feedbacks in Devflow UI.

2026-09-04 10:32:04 UTC ℹ️ Start processing command /merge

2026-09-04 10:32:07 UTC ❌ MergeQueue

PR already in the queue with status waiting

bric3 · 2026-09-04T12:51:28Z

/merge

gh-worker-devflow-routing-ef8351 · 2026-09-04T12:51:32Z

View all feedbacks in Devflow UI.

2026-09-04 12:51:32 UTC ℹ️ Start processing command /merge

2026-09-04 12:51:38 UTC ℹ️ MergeQueue: pull request added to the queue

The expected merge time in master is approximately 1h (p90).

2026-09-04 13:40:56 UTC ❌ MergeQueue: The build pipeline contains failing jobs for this merge request

Build pipeline has failing jobs for 5b9a957:

test_inst: [semeru17, 5/8]

⚠️ Do NOT retry failed jobs directly (why?).

What to do next?

Investigate the failures and when ready, re-add your pull request to the queue!
If your PR checks are green, try to rebase/merge. It might be because the CI run is a bit old.
Any question, go check the FAQ.

Details

Since those jobs are not marked as being allowed to fail, the pipeline will most likely fail.
Therefore, and to allow other builds to be processed, this merge request has been rejected and the pipeline got canceled.

bric3 · 2026-09-04T13:46:37Z

/merge

gh-worker-devflow-routing-ef8351 · 2026-09-04T13:46:41Z

View all feedbacks in Devflow UI.

2026-09-04 13:46:41 UTC ℹ️ Start processing command /merge

2026-09-04 13:46:48 UTC ℹ️ MergeQueue: pull request added to the queue

The expected merge time in master is approximately 1h (p90).

2026-09-04 15:47:32 UTC ❌ MergeQueue: The build pipeline has timeout

The merge request has been interrupted because the build 6729387127231966687 took longer than expected. The current limit for the base branch 'master' is 120 minutes.

bric3 added 4 commits August 25, 2026 14:19

refactor: migrate Kotlin test setup to convention

9fecc27

Keep shared Kotlin and default Groovy test wiring behind plugin callbacks.

refactor: wire Kotlin outputs for test source sets

a8b24c4

Cover JVM test suites and test fixtures through Gradle source-set APIs.

refactor: apply Java convention from remaining projects

65c0c56

refactor: make Java convention the composition root

ebc24bc

bric3 added tag: no release notes Changes to exclude from release notes type: refactoring comp: tooling Build & Tooling tag: ai generated Largely based on code generated by an AI or LLM labels Aug 28, 2026

bric3 marked this pull request as ready for review August 28, 2026 13:59

bric3 requested a review from a team as a code owner August 28, 2026 13:59

bric3 requested review from amarziali and removed request for a team August 28, 2026 13:59

amarziali approved these changes Aug 28, 2026

View reviewed changes

This comment has been minimized.

Sign in to view

datadog-prod-us1-6 Bot reviewed Aug 28, 2026

View reviewed changes

Base automatically changed from refactor/apply-java-convention-remaining-projects to master September 4, 2026 09:38

chore: merge master into Java convention branch

56f0ecf

Navigation Menu

Uh oh!

Make Java convention the composition root - #12333

Make Java convention the composition root#12333
bric3 wants to merge 5 commits into
masterfrom
bdu/java-convention-composition-root

bric3 commented Aug 28, 2026 •

edited

Loading

Uh oh!

This comment has been minimized.

datadog-prod-us1-6 Bot left a comment

Uh oh!

dd-octo-sts Bot commented Aug 28, 2026 •

edited

Loading

Uh oh!

bric3 commented Sep 4, 2026

Uh oh!

gh-worker-devflow-routing-ef8351 Bot commented Sep 4, 2026 •

edited

Loading

Uh oh!

bric3 commented Sep 4, 2026

Uh oh!

gh-worker-devflow-routing-ef8351 Bot commented Sep 4, 2026 •

edited

Loading

Uh oh!

bric3 commented Sep 4, 2026

Uh oh!

gh-worker-devflow-routing-ef8351 Bot commented Sep 4, 2026 •

edited

Loading

Uh oh!

bric3 commented Sep 4, 2026

Uh oh!

gh-worker-devflow-routing-ef8351 Bot commented Sep 4, 2026 •

edited

Loading

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Uh oh!

Conversation

bric3 commented Aug 28, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

What Does This Do

Motivation

Contributor Checklist

Uh oh!

This comment has been minimized.

datadog-prod-us1-6 Bot left a comment

Choose a reason for hiding this comment

Uh oh!

dd-octo-sts Bot commented Aug 28, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

🟢 Java Benchmark SLOs — All performance SLOs passed

Uh oh!

bric3 commented Sep 4, 2026

Uh oh!

gh-worker-devflow-routing-ef8351 Bot commented Sep 4, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

What to do next?

Uh oh!

bric3 commented Sep 4, 2026

Uh oh!

gh-worker-devflow-routing-ef8351 Bot commented Sep 4, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

bric3 commented Sep 4, 2026

Uh oh!

gh-worker-devflow-routing-ef8351 Bot commented Sep 4, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

What to do next?

Uh oh!

bric3 commented Sep 4, 2026

Uh oh!

gh-worker-devflow-routing-ef8351 Bot commented Sep 4, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

bric3 commented Aug 28, 2026 •

edited

Loading

dd-octo-sts Bot commented Aug 28, 2026 •

edited

Loading

gh-worker-devflow-routing-ef8351 Bot commented Sep 4, 2026 •

edited

Loading

gh-worker-devflow-routing-ef8351 Bot commented Sep 4, 2026 •

edited

Loading

gh-worker-devflow-routing-ef8351 Bot commented Sep 4, 2026 •

edited

Loading

gh-worker-devflow-routing-ef8351 Bot commented Sep 4, 2026 •

edited

Loading