Skip to content

Conversation

@martincostello
Copy link
Member

Update benchmark results ahead of merging #2664.

Update benchmark results ahead of merging #2664.
@Copilot Copilot AI review requested due to automatic review settings August 22, 2025 08:36
@martincostello martincostello enabled auto-merge (rebase) August 22, 2025 08:36
Copy link

Copilot AI left a 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 results across the Polly.Core codebase, reflecting performance measurements run on a newer environment ahead of merging PR #2664.

  • Updates system environment information (Windows build and .NET SDK versions)
  • Refreshes performance metrics showing generally improved execution times
  • Updates benchmark data across all core Polly strategies and components

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
TimeoutBenchmark-report-github.md Updated timeout strategy benchmarks with improved V7/V8 performance metrics
TelemetryBenchmark-report-github.md Refreshed telemetry overhead measurements with reduced execution times
RetryBenchmark-report-github.md Updated retry strategy performance data showing faster execution
ResiliencePipelineProviderBenchmark-report-github.md Improved pipeline provider retrieval benchmarks
ResiliencePipelineBenchmark-report-github.md Updated core resilience pipeline execution metrics
RateLimiterBenchmark-report-github.md Refreshed rate limiting strategy performance data
PredicateBenchmark-report-github.md Updated predicate evaluation benchmark results
PipelineBenchmark-report-github.md Multi-component pipeline performance improvements
MultipleStrategiesBenchmark-report-github.md Complex strategy composition benchmark updates
HedgingBenchmark-report-github.md Hedging strategy performance improvements
GenericOverheadBenchmark-report-github.md Generic vs non-generic API overhead measurements
DelegatingComponentBenchmark-report-github.md Component delegation performance updates
CreationBenchmark-report-github.md Strategy creation overhead benchmarks
CompositeComponentBenchmark-report-github.md Composite component execution performance
CircuitBreakerOpenedBenchmark-report-github.md Circuit breaker opened state benchmark updates
CircuitBreakerBenchmark-report-github.md Circuit breaker strategy performance improvements
BridgeBenchmark-report-github.md Bridge component overhead measurements

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@codecov
Copy link

codecov bot commented Aug 22, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 96.23%. Comparing base (f52f37f) to head (e535405).
⚠️ Report is 1 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2712   +/-   ##
=======================================
  Coverage   96.23%   96.23%           
=======================================
  Files         311      311           
  Lines        7322     7322           
  Branches     1012     1012           
=======================================
  Hits         7046     7046           
  Misses        222      222           
  Partials       54       54           
Flag Coverage Δ
linux 96.23% <ø> (ø)
macos 96.23% <ø> (ø)
windows 96.22% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@martincostello martincostello merged commit 5603fa0 into main Aug 22, 2025
28 checks passed
@martincostello martincostello deleted the update-benchmark-results branch August 22, 2025 08:44
This was referenced Aug 22, 2025
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.

1 participant