Skip to content

Refactor benchmarks #1203

New issue

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

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

Already on GitHub? # to your account

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

fdcastel
Copy link
Member

Based on #1200 for now. I will rebase it with latest master in the future.

  • Rename project Perf to FirebirdSql.Data.FirebirdClient.Benchmarks.
  • Update project to use .NET8.
  • Upgrade BenchmarkDotNet to version 0.14.0.
  • Update baseline nuget package to v10.3.1.
  • Add /BenchmarkDotNet.Artifacts folder to .gitignore.
  • Pass command-line arguments to BenchmarkDotNet engine.
  • Apply SQL Formatting. Use raw strings.
  • Add script run-benchmark.ps1.

AddJob(baseJob.WithCustomBuildConfiguration("ReleaseNuGet").WithId("NuGet").AsBaseline());

AddValidator(BaselineValidator.FailOnError);
AddValidator(JitOptimizationsValidator.FailOnError);
Copy link
Contributor

Choose a reason for hiding this comment

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

If full JSON (*full.json) reports are exported then benchmarks results can be compared historically using ResultsComparer.

AddExporter(JsonExporter.Full);

Copy link
Contributor

Choose a reason for hiding this comment

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

Never mind, I see how it is supposed to work now!

fdcastel added 5 commits May 17, 2025 22:16
- Rename project 'Perf' to 'FirebirdSql.Data.FirebirdClient.Benchmarks'.
- Update project to use .net8.
- Upgrade BenchmarkDotNet to version 0.14.0.
- Update baseline nuget package to v10.3.1.
- Add /BenchmarkDotNet.Artifacts to .gitignore.
- Pass command-line arguments to BenchmarkDotNet engine.
- Apply SQL Formatting. Use raw strings.
- Add script run-benchmark.ps1.
@fdcastel fdcastel force-pushed the refactor-benchmarks branch from bd2e3a0 to e5e4075 Compare May 18, 2025 01:19
@fdcastel
Copy link
Member Author

Rebase with master.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants