Skip to content

Enhancements to SlidingCache<> for Improved Thread Safety and Performance#770

Merged
StefH merged 3 commits intozzzprojects:masterfrom
TWhidden:master
Feb 3, 2024
Merged

Enhancements to SlidingCache<> for Improved Thread Safety and Performance#770
StefH merged 3 commits intozzzprojects:masterfrom
TWhidden:master

Conversation

@TWhidden
Copy link
Copy Markdown
Contributor

@TWhidden TWhidden commented Feb 2, 2024

See #769 for description of changes;

Comment thread src/System.Linq.Dynamic.Core/Util/Cache/CacheConfig.cs Outdated
Comment thread test/System.Linq.Dynamic.Core.Tests/Util/Cache/SlidingCacheTests.cs
Comment thread test/System.Linq.Dynamic.Core.Tests/Util/Cache/SlidingCacheTests.cs
Comment thread test/System.Linq.Dynamic.Core.Tests/Util/Cache/SlidingCacheTests.cs Outdated
@TWhidden TWhidden requested a review from StefH February 3, 2024 06:54
Copy link
Copy Markdown
Collaborator

@StefH StefH left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

just add a comment to the unit test file

@TWhidden TWhidden requested a review from StefH February 3, 2024 07:52
@StefH StefH added the feature label Feb 3, 2024
Copy link
Copy Markdown
Collaborator

@StefH StefH left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

.

@StefH StefH merged commit 7e3b8ed into zzzprojects:master Feb 3, 2024
@StefH
Copy link
Copy Markdown
Collaborator

StefH commented Feb 3, 2024

I did a small fix on the unit tests.
e6bc36e

And I'll release a new preview soon.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Development

Successfully merging this pull request may close these issues.

2 participants