Skip to content

Expose ValueWrapper to differentiate between cached null values and absent cache mapping. #2785

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

Closed
wants to merge 4 commits into from

Conversation

mp911de
Copy link
Member

@mp911de mp911de commented Nov 23, 2023

We now use ValueWrapper to differentiate in the async API between cache misses and cached null values.

Closes #2783

…bsent cache mapping.

We now use ValueWrapper to differentiate in the async API between cache misses and cached null values.
Reformat code.
@mp911de mp911de added the type: enhancement A general enhancement label Nov 23, 2023
@mp911de mp911de added this to the 3.2.1 (2023.1.1) milestone Nov 23, 2023
christophstrobl pushed a commit that referenced this pull request Dec 14, 2023
…bsent cache mapping.

We now use ValueWrapper to differentiate in the async API between cache misses and cached null values.

Closes: #2783
Original Pull Request: #2785
christophstrobl pushed a commit that referenced this pull request Dec 14, 2023
Reformat code.

Original Pull Request: #2785
christophstrobl pushed a commit that referenced this pull request Dec 14, 2023
…bsent cache mapping.

We now use ValueWrapper to differentiate in the async API between cache misses and cached null values.

Closes: #2783
Original Pull Request: #2785
christophstrobl pushed a commit that referenced this pull request Dec 14, 2023
Reformat code.

Original Pull Request: #2785
christophstrobl added a commit that referenced this pull request Dec 14, 2023
Delegate value loading to existing method to remove duplicate code paths.

Original Pull Request: #2785
@christophstrobl christophstrobl deleted the issue/2783 branch December 14, 2023 07:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: enhancement A general enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

RedisCache should expose ValueWrapper for cached null values returned from retrieve(key)
2 participants