Migrate benchmarks to Merge Queue by sarahchen6 · Pull Request #10526 · DataDog/dd-trace-java · GitHub
Skip to content

Migrate benchmarks to Merge Queue#10526

Merged
gh-worker-dd-mergequeue-cf854d[bot] merged 4 commits into
masterfrom
sarahchen6/move-benchmarks-to-mq
Apr 29, 2026
Merged

Migrate benchmarks to Merge Queue#10526
gh-worker-dd-mergequeue-cf854d[bot] merged 4 commits into
masterfrom
sarahchen6/move-benchmarks-to-mq

Conversation

@sarahchen6

@sarahchen6 sarahchen6 commented Feb 4, 2026

Copy link
Copy Markdown
Contributor

What Does This Do

Run startup, load, and dacapo benchmarks on the merge queue instead of for every PR push. Allow these benchmarks to be run manually in the corresponding Gitlab pipeline as well.

Motivation

By migrating these benchmarks to the merge queue, we can greatly reduce CI runtime (from ~60 min to ~30 min) and support faster development experiences. At the same time, however, we don't want to lose benchmarking ability nor merge in unknown regressions. This is addressed by still allowing manual benchmark runs on PRs and the check-big-regressions gate.

Additional Notes

The check-big-regressions check will still be in place and fail on regressions of 10+%; however, it will not catch small or unstable regressions. Developers should still manually run benchmarks once done iterating to see results.

See this 30 min pipeline run with benchmarks on MQ, and this 52 min pipeline run with manually run benchmarks.

Contributor Checklist

Jira ticket: [PROJ-IDENT]

@sarahchen6 sarahchen6 added type: enhancement Enhancements and improvements tag: no release notes Changes to exclude from release notes comp: tooling Build & Tooling labels Feb 4, 2026
@sarahchen6 sarahchen6 force-pushed the sarahchen6/move-benchmarks-to-mq branch from c3c9539 to 32b3af7 Compare April 28, 2026 19:27
@sarahchen6 sarahchen6 changed the title Migrate benchmarks to MergeQueue Migrate benchmarks to Merge Queue Apr 29, 2026
@pr-commenter

pr-commenter Bot commented Apr 29, 2026

Copy link
Copy Markdown

@sarahchen6 sarahchen6 marked this pull request as ready for review April 29, 2026 17:07
@sarahchen6 sarahchen6 requested a review from a team as a code owner April 29, 2026 17:07
@sarahchen6 sarahchen6 requested review from AlexeyKuznetsov-DD and randomanderson and removed request for a team April 29, 2026 17:07

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 3585ce7875

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

Comment thread .gitlab/benchmarks.yml Outdated
@sarahchen6

Copy link
Copy Markdown
Contributor Author

/merge

@gh-worker-devflow-routing-ef8351

gh-worker-devflow-routing-ef8351 Bot commented Apr 29, 2026

Copy link
Copy Markdown

@gh-worker-dd-mergequeue-cf854d gh-worker-dd-mergequeue-cf854d Bot merged commit 52c2ac6 into master Apr 29, 2026
567 of 568 checks passed
@gh-worker-dd-mergequeue-cf854d gh-worker-dd-mergequeue-cf854d Bot deleted the sarahchen6/move-benchmarks-to-mq branch April 29, 2026 19:37
@github-actions github-actions Bot added this to the 1.62.0 milestone Apr 29, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

comp: tooling Build & Tooling tag: no release notes Changes to exclude from release notes type: enhancement Enhancements and improvements

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants