Skip to content

Performance badges #17

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 1 commit into from
Aug 15, 2025
Merged

Performance badges #17

merged 1 commit into from
Aug 15, 2025

Conversation

Hoodie-Huuuuu
Copy link
Collaborator

@Hoodie-Huuuuu Hoodie-Huuuuu commented Aug 12, 2025

This PR introduces an automated performance benchmark comparison between the master branch and the Pull Request branch, with the results automatically posted as a PR comment.

  • CI image updated to 0.0.2 with Python support.
  • Performance job in test.yml:
    • For PRs: downloads latest master benchmarks, runs current benchmarks, compares via compare_benchmarks.py, posts markdown report.
    • For master pushes: the same actions as for PRs + uploads updated master benchmark artifacts.
  • New tool:
  • build/ci/compare_benchmarks.py — parses and compares Go benchmark metrics, highlights improvements ⚡️ and regressions 💔, outputs markdown.

@Hoodie-Huuuuu Hoodie-Huuuuu force-pushed the performance-badges branch 2 times, most recently from 4ca129e to 9f49d6b Compare August 12, 2025 09:11
@tarantool tarantool deleted a comment from github-actions bot Aug 12, 2025
Copy link
Collaborator

@hackallcode hackallcode left a comment

Choose a reason for hiding this comment

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

Thanks for the patch, it's useful!
Let's change a couple of things.

@tarantool tarantool deleted a comment from github-actions bot Aug 12, 2025
Copy link
Collaborator

@hackallcode hackallcode left a comment

Choose a reason for hiding this comment

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

Great!

# Conflicts:
#	.github/workflows/ci.yml
Copy link

Warning

No test results found for master branch. Please run workflow on master first to compare results.

Perf tests report: performance-badges

Benchmark B/s values/s
Partitioning (cpu=4) ? → 90.69M ⚠️ ? → 6.65M ⚠️
CSVWriter (cpu=4) ? → 47.05M ⚠️ ? → 3.45M ⚠️
ParquetWriter (cpu=4) ? → 206.59M ⚠️ ? → 15.15M ⚠️
HTTPWriter (cpu=4) ? → 116.32M ⚠️ ? → 8.53M ⚠️
TCSWriter (cpu=4) ? → 115.62M ⚠️ ? → 8.48M ⚠️
Integer Random (bits=32, cpu=4) ? → 101.27M ⚠️ ? → 24.15M ⚠️
Integer Ordered (bits=32, cpu=4) ? → 94.43M ⚠️ ? → 22.52M ⚠️
Float Random (bits=32, cpu=4) ? → 103.53M ⚠️ ? → 24.68M ⚠️
String Random (length=16, cpu=4) ? → 96.79M ⚠️ ? → 5.77M ⚠️
Text Random (length=128, cpu=4) ? → 153.32M ⚠️ ? → 1.14M ⚠️
UUID Random (cpu=4) ? → 307.40M ⚠️ ? → 18.32M ⚠️
DateTime Random (cpu=4) ? → 301.00M ⚠️ ? → 17.94M ⚠️
Enum Random (cpu=4) ? → 536.97M ⚠️ ? → 32.01M ⚠️
Ranges Random (cpu=4) ? → 396.14M ⚠️ ? → 23.61M ⚠️

@Hoodie-Huuuuu Hoodie-Huuuuu merged commit 732dc05 into master Aug 15, 2025
14 checks passed
@Hoodie-Huuuuu Hoodie-Huuuuu deleted the performance-badges branch August 15, 2025 07:22
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.

2 participants