Skip to content
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

HybridCache - fix RemoveKeyAsync incorrectly calling RemoveByTagAsync #56719

Merged
merged 1 commit into from
Jul 10, 2024

Conversation

mgravell
Copy link
Member

@mgravell mgravell commented Jul 10, 2024

  • fix RemoveKeyAsync incorrectly calling RemoveByTagAsync
  • add Remove tests
  • drive-by fix async void tests (probably auto-IDE)

Fixes #56581

- RemoveKeyAsync incorrectly calling RemoveByTagAsync
- add Remove tests
- drive-by fix async void tests (probably auto-IDE)
@dotnet-issue-labeler dotnet-issue-labeler bot added the needs-area-label Used by the dotnet-issue-labeler to label those issues which couldn't be triaged automatically label Jul 10, 2024
mgravell added a commit to mgravell/runtime that referenced this pull request Jul 10, 2024
@mgravell mgravell enabled auto-merge (squash) July 10, 2024 14:47
mgravell added a commit to mgravell/runtime that referenced this pull request Jul 10, 2024
@mgravell mgravell merged commit 7473506 into main Jul 10, 2024
26 checks passed
@mgravell mgravell deleted the marc/hc-remove-async branch July 10, 2024 18:00
@dotnet-policy-service dotnet-policy-service bot added this to the 9.0-preview7 milestone Jul 10, 2024
jozkee added a commit to dotnet/runtime that referenced this pull request Jul 22, 2024
…103103)

* HybridCache migration from aspnet

* use cancellationToken instead of token

* reapply dotnet/aspnetcore#56719

* Update src/libraries/Microsoft.Extensions.Caching.Abstractions/src/Hybrid/HybridCache.cs

Co-authored-by: David Cantú <[email protected]>

* Update src/libraries/Microsoft.Extensions.Caching.Abstractions/src/Hybrid/HybridCache.cs

Co-authored-by: David Cantú <[email protected]>

* Update src/libraries/Microsoft.Extensions.Caching.Abstractions/src/Hybrid/HybridCache.cs

Co-authored-by: David Cantú <[email protected]>

* prefer throw null

* use IEnumerable<string> instead of IReadOnlyCollection<string>

* remove suppressions

* Update src/libraries/Microsoft.Extensions.Caching.Abstractions/src/Hybrid/IHybridCacheSerializer.cs

Co-authored-by: Stephen Toub <[email protected]>

* Update src/libraries/Microsoft.Extensions.Caching.Abstractions/src/Hybrid/HybridCacheEntryOptions.cs

Co-authored-by: Stephen Toub <[email protected]>

* Update src/libraries/Microsoft.Extensions.Caching.Abstractions/src/Hybrid/HybridCacheEntryOptions.cs

Co-authored-by: Stephen Toub <[email protected]>

* PR nits

---------

Co-authored-by: David Cantú <[email protected]>
Co-authored-by: Stephen Toub <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs-area-label Used by the dotnet-issue-labeler to label those issues which couldn't be triaged automatically
Projects
None yet
Development

Successfully merging this pull request may close these issues.

HybridCache.RemoveAsync calls RemoveByTagAsync instead of RemoveAsync
2 participants