Hedging strategy also deep-copies context for primary execution

[martintmk]

Pull Request

The issue or feature being addressed

#1727

Details on the issue fix or feature implementation

As per #1727 there was inconsistency on how the hedging contexts for primary executions are handled. Instead of creating deep-clone, the hedging strategy re-used the original context.

With this change, the handling is the same for both primary and hedged actions. Hedging strategy always creates a deep clone in all scenarios.

This also simplifies the implementation and handling of contexts. 

Confirm the following

• I started this PR by branching from the head of the default branch
• I have targeted the PR to merge into the default branch
• I have included unit tests for the issue/feature
• I have successfully run a local build

[martincostello]

Is there any impact to benchmarks (+ve or -ve) from this change?

[martintmk]

Is there any impact to benchmarks (+ve or -ve) from this change?

I don't think so, but let me re-run just in case.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (41d4e8d) 84.56% compared to head (c7c97e1) 84.52%.
Report is 1 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1754      +/-   ##
==========================================
- Coverage   84.56%   84.52%   -0.05%     
==========================================
  Files         307      306       -1     
  Lines        6790     6772      -18     
  Branches     1043     1042       -1     
==========================================
- Hits         5742     5724      -18     
  Misses        839      839              
  Partials      209      209              

Flag	Coverage Δ
linux	84.52% <100.00%> (-0.05%)	⬇️
macos	84.52% <100.00%> (-0.05%)	⬇️
windows	?

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

Files	Coverage Δ
...Core/Hedging/Controller/HedgingExecutionContext.cs	100.00% <100.00%> (ø)
src/Polly.Core/Hedging/Controller/TaskExecution.cs	100.00% <100.00%> (ø)
src/Polly.Core/ResilienceContext.cs	100.00% <100.00%> (ø)
src/Polly.Core/ResilienceProperties.cs	100.00% <ø> (ø)

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

[martincostello]

@martintmk Would you consider this change to be a patch or a minor change?

If we take the change for AoT too, then can that be 8.0.1 or does this change make it 8.1.0?

I'm thinking 8.0.1, but if you consider this minor rather than patch then we can do an 8.1.0 and the .NET 8 release will become 8.2.0.

[martintmk]

We had some API additions in #1687 so I would say we should do 8.1.0.