-
Notifications
You must be signed in to change notification settings - Fork 3
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
Fix RequestCollapserEngine caching lazy with exceptions as value indefinitely #6
Fix RequestCollapserEngine caching lazy with exceptions as value indefinitely #6
Conversation
…ill result in the RequestCollapser's Engine to cache the exception indefinitely.
src/Polly.Contrib.DuplicateRequestCollapser.Specs/RequestCollapserTResultSpecsBase.cs
Outdated
Show resolved
Hide resolved
src/Polly.Contrib.DuplicateRequestCollapser.Specs/RequestCollapserTResultSpecsBase.cs
Outdated
Show resolved
Hide resolved
src/Polly.Contrib.DuplicateRequestCollapser.Specs/RequestCollapserTResultSpecsBase.cs
Outdated
Show resolved
Hide resolved
src/Polly.Contrib.DuplicateRequestCollapser.Specs/RequestCollapserTResultSpecsBase.cs
Outdated
Show resolved
Hide resolved
src/Polly.Contrib.DuplicateRequestCollapser.Specs/RequestCollapserTResultSpecs.cs
Show resolved
Hide resolved
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.
🕐
@reisenberger Would love your input |
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.
How do we get an updated nuget built? |
@veleek @reisenberger Either of you able to do the nuget voodoo for this? I think v. 0.3.0 would be the next version. |
I do not. I'm assuming everybody in the https://github.com/orgs/Polly-Contrib/people group have access. |
@jjxtra I'll help you out with the package deployment. Taking a look now. |
@joelhulen Thank you so much for your assistance! |
Of course! Thank you for your contribution :) |
When the valueFactory inside a Lazy object throws an exception, the collapser engine is not properly cleaning it up from the ConcurrentDictionary. This results in the RequestCollapserPolicy caching the result indefinitely.