Skip to content

GODRIVER-3102: Perf comp PR comment pipeline #2149

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 16 commits into from
Aug 6, 2025

Conversation

zhouselena
Copy link
Contributor

@zhouselena zhouselena commented Jul 30, 2025

GODRIVER-3102

Summary

This is the pipeline for the perf comparison PR comment. It always outputs the results to the logs. If the commit being tested belongs to a PR, the pipeline will post a new comment or update an existing one on the PR with the performance comparison details.

Background & Motivation

This is so developers can see changes directly in the PR without needing to go through the Evergreen Parsley logs. Also, developers can proactively see how their commits affect performance before merging to mainline.

@mongodb-drivers-pr-bot mongodb-drivers-pr-bot bot added the review-priority-low Low Priority PR for Review: within 3 business days label Jul 30, 2025
Copy link
Contributor

API Change Report

No changes found!

@mongodb mongodb deleted a comment from mongodb-drivers-pr-bot bot Jul 30, 2025
@zhouselena zhouselena marked this pull request as ready for review July 30, 2025 17:33
@zhouselena zhouselena requested a review from a team as a code owner July 30, 2025 17:33
Copy link
Contributor

mongodb-drivers-pr-bot bot commented Jul 30, 2025

👋 GoDriver Performance

Commit SHA: ce710ae

The following benchmark tests for version 688bf0e5a985180007d026db had statistically significant changes (i.e., |z-score| > 1.96):

Benchmark Measurement % Change Patch Value Stable Region H-Score Z-Score
BenchmarkMultiInsertLargeDocument total_time_seconds 4.4242 1.1783 Avg: 1.1284
Med: 1.1315
Stdev: 0.0248
0.7411 2.0164
BenchmarkMultiInsertLargeDocument total_mem_allocs 4.1297 1951.0000 Avg: 1873.6250
Med: 1884.0000
Stdev: 21.4539
0.8807 3.6066
BenchmarkMultiInsertSmallDocument allocated_bytes_per_op 3.4724 2811.0000 Avg: 2716.6667
Med: 2709.5000
Stdev: 39.8179
0.8057 2.3691
BenchmarkMultiInsertLargeDocument allocs_per_op 1.8519 50.0000 Avg: 49.0909
Med: 49.0000
Stdev: 0.3015
0.9091 3.0151
BenchmarkSmallDocInsertOne allocated_bytes_per_op -0.3003 5599.0000 Avg: 5615.8646
Med: 5618.5000
Stdev: 6.6390
0.8099 -2.5402

For a comprehensive view of all microbenchmark results for this PR's commit, please check out the Evergreen perf task for this patch.

@mongodb mongodb deleted a comment from mongodb-drivers-pr-bot bot Jul 30, 2025
Copy link
Collaborator

@matthewdale matthewdale left a comment

Choose a reason for hiding this comment

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

Overall looks great! A few suggestions and questions.

Copy link
Contributor

mongodb-drivers-pr-bot bot commented Aug 1, 2025

🧪 Performance Results

Commit SHA: 2355f33

There were no significant changes to the performance to report for version 68937892f0691c0007204893.

Benchmark Measurement % Change Patch Value Stable Region H-Score Z-Score

For a comprehensive view of all microbenchmark results for this PR's commit, please check out the Evergreen perf task for this patch.

@zhouselena zhouselena requested a review from matthewdale August 4, 2025 18:18
Copy link
Collaborator

@matthewdale matthewdale left a comment

Choose a reason for hiding this comment

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

Looks good! 👍

Copy link
Collaborator

@qingyang-hu qingyang-hu left a comment

Choose a reason for hiding this comment

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

LGTM 🎉

Copy link
Member

@prestonvasquez prestonvasquez left a comment

Choose a reason for hiding this comment

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

Great work 🔧 🔢

@zhouselena zhouselena merged commit 4ee9c12 into mongodb:master Aug 6, 2025
31 of 36 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
review-priority-low Low Priority PR for Review: within 3 business days
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants