-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
Update benchmarks #2713
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
Update benchmarks #2713
Conversation
Update benchmarks after merging changes from #2664.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #2713 +/- ##
=======================================
Coverage 96.12% 96.12%
=======================================
Files 309 309
Lines 7118 7118
Branches 1006 1006
=======================================
Hits 6842 6842
Misses 222 222
Partials 54 54
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
5603fa0 vs. a14508b
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR updates benchmark reports to reflect performance changes after merging improvements from PR #2664. The changes show updated performance metrics including execution times, memory allocations, and ratios across various Polly Core components.
- Updated benchmark results across 18 benchmark files
- Performance improvements shown in most benchmark scenarios
- Memory allocation reductions in several test cases
Reviewed Changes
Copilot reviewed 17 out of 17 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| Polly.Core.Benchmarks.TimeoutBenchmark-report-github.md | Updated timeout benchmark results showing improved performance for both V7 and V8 |
| Polly.Core.Benchmarks.TelemetryBenchmark-report-github.md | Updated telemetry benchmark results with improved execution times |
| Polly.Core.Benchmarks.RetryBenchmark-report-github.md | Updated retry benchmark results showing performance changes |
| Polly.Core.Benchmarks.ResiliencePipelineProviderBenchmark-report-github.md | Updated pipeline provider benchmark results |
| Polly.Core.Benchmarks.ResiliencePipelineBenchmark-report-github.md | Updated resilience pipeline benchmark results |
| Polly.Core.Benchmarks.RateLimiterBenchmark-report-github.md | Updated rate limiter benchmark results |
| Polly.Core.Benchmarks.PredicateBenchmark-report-github.md | Updated predicate benchmark results |
| Polly.Core.Benchmarks.PipelineBenchmark-report-github.md | Updated pipeline benchmark results across multiple component configurations |
| Polly.Core.Benchmarks.MultipleStrategiesBenchmark-report-github.md | Updated multiple strategies benchmark results |
| Polly.Core.Benchmarks.HedgingBenchmark-report-github.md | Updated hedging benchmark results showing significant performance improvements |
| Polly.Core.Benchmarks.GenericOverheadBenchmark-report-github.md | Updated generic overhead benchmark results |
| Polly.Core.Benchmarks.DelegatingComponentBenchmark-report-github.md | Updated delegating component benchmark results |
| Polly.Core.Benchmarks.CreationBenchmark-report-github.md | Updated creation benchmark results |
| Polly.Core.Benchmarks.CompositeComponentBenchmark-report-github.md | Updated composite component benchmark results |
| Polly.Core.Benchmarks.CircuitBreakerOpenedBenchmark-report-github.md | Updated circuit breaker opened benchmark results |
| Polly.Core.Benchmarks.CircuitBreakerBenchmark-report-github.md | Updated circuit breaker benchmark results |
| Polly.Core.Benchmarks.BridgeBenchmark-report-github.md | Updated bridge benchmark results |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
Update benchmarks after merging changes from #2664.
Just need to do some number crunching on the ratios to do a summary table of the changes.