Skip to content

Conversation

@mgravell
Copy link
Contributor

@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)
@ghost ghost 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
@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

3 participants