Skip to content

Fix the benchmarks - #272

Merged
zhengyu123 merged 2 commits into
mainfrom
zgu/fix-benchmark
Sep 12, 2025
Merged

Fix the benchmarks#272
zhengyu123 merged 2 commits into
mainfrom
zgu/fix-benchmark

Conversation

@zhengyu123

@zhengyu123zhengyu123 commented Sep 12, 2025

Copy link
Copy Markdown
Contributor

What does this PR do?:
This is the java-profiler side of fixing for benchmarking.

Motivation:
Enabling benchmarking on PR

Additional Notes:

How to test the change?:
Downstream build still triggered, no regression.

For Datadog employees:

  • If this PR touches code that signs or publishes builds or packages, or handles
    credentials of any kind, I've requested a review from @DataDog/security-design-and-guidance.
  • This PR doesn't touch any of that.
  • JIRA: PROF-11835 Fix the benchmarks

Unsure? Have a question? Request a review!

@zhengyu123
zhengyu123 marked this pull request as ready for review September 12, 2025 12:43
Comment thread.gitlab-ci.yml
@pr-commenter

pr-commenterBot commented Sep 12, 2025

Copy link
Copy Markdown

Benchmarks

Parameters

BaselineCandidate
configbaselinecandidate
ddprof1.31.01.32.0-zgu_fix-benchmark-SNAPSHOT
See matching parameters
BaselineCandidate
allocoffoff
cpuonon
iterations55
java
memleakoffoff
modescpu,wallcpu,wall
wallonon

Summary

Found 0 performance improvements and 0 performance regressions! Performance is the same for 18 metrics, 1 unstable metrics.

@zhengyu123
zhengyu123 merged commit bfc63e0 into mainSep 12, 2025
94 checks passed
@zhengyu123
zhengyu123 deleted the zgu/fix-benchmark branch September 12, 2025 13:50
@github-actionsgithub-actionsBot added this to the 1.32.0 milestone Sep 12, 2025
Sign up for freeto 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.

3 participants

@zhengyu123@jbachorik@MattAlp