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

Add support for asynchronous retrieval from RedisCache #2650

Closed
christophstrobl opened this issue Jul 25, 2023 · 0 comments
Closed

Add support for asynchronous retrieval from RedisCache #2650

christophstrobl opened this issue Jul 25, 2023 · 0 comments
Assignees
Labels
type: enhancement A general enhancement

Comments

@christophstrobl
Copy link
Member

See: spring-projects/spring-framework#17920
Related issue: #1541

@christophstrobl christophstrobl added the type: enhancement A general enhancement label Jul 25, 2023
@mp911de mp911de changed the title Add reactive types support to RedisCacheManager. Add reactive types support to RedisCacheManager Aug 9, 2023
@mp911de mp911de changed the title Add reactive types support to RedisCacheManager Add reactive types support to RedisCacheManager Aug 9, 2023
jxblum added a commit to jxblum/spring-data-redis that referenced this issue Aug 15, 2023
jxblum added a commit to jxblum/spring-data-redis that referenced this issue Sep 21, 2023
jxblum added a commit to jxblum/spring-data-redis that referenced this issue Sep 21, 2023
jxblum added a commit to jxblum/spring-data-redis that referenced this issue Sep 21, 2023
jxblum added a commit to jxblum/spring-data-redis that referenced this issue Sep 22, 2023
jxblum added a commit to jxblum/spring-data-redis that referenced this issue Sep 23, 2023
@jxblum jxblum changed the title Add reactive types support to RedisCacheManager Add reactive types support to RedisCache Sep 23, 2023
jxblum added a commit to jxblum/spring-data-redis that referenced this issue Sep 25, 2023
jxblum added a commit to jxblum/spring-data-redis that referenced this issue Sep 28, 2023
jxblum added a commit to jxblum/spring-data-redis that referenced this issue Oct 3, 2023
jxblum added a commit to jxblum/spring-data-redis that referenced this issue Oct 3, 2023
jxblum added a commit to jxblum/spring-data-redis that referenced this issue Oct 3, 2023
jxblum added a commit to jxblum/spring-data-redis that referenced this issue Oct 3, 2023
jxblum added a commit to jxblum/spring-data-redis that referenced this issue Oct 4, 2023
jxblum added a commit to jxblum/spring-data-redis that referenced this issue Oct 4, 2023
…ableFuture<T>> Cache operations without Jedis support.

Closes spring-projects#2650
@mp911de mp911de linked a pull request Oct 10, 2023 that will close this issue
@mp911de mp911de changed the title Add reactive types support to RedisCache Adds support for asynchronous retrieval from RedisCache Oct 10, 2023
jxblum added a commit to jxblum/spring-data-redis that referenced this issue Oct 12, 2023
Cleanup ambigious, unreadable, unstructured and overly complex code in DefaultRedisCacheWriter.

Split RedisCacheTests.retrieveCacheValueUsingJedis() test logic by test case.

Edit Javadoc.

See spring-projects#2650
Original pull request: spring-projects#2717
jxblum added a commit to jxblum/spring-data-redis that referenced this issue Oct 12, 2023
jxblum added a commit to jxblum/spring-data-redis that referenced this issue Oct 12, 2023
Cleanup ambigious, unreadable, unstructured and overly complex code in DefaultRedisCacheWriter.

Split RedisCacheTests.retrieveCacheValueUsingJedis() test logic by test case.

Edit Javadoc.

See spring-projects#2650
Original pull request: spring-projects#2717
jxblum added a commit to jxblum/spring-data-redis that referenced this issue Oct 12, 2023
Cleanup ambigious, unreadable, unstructured and complex logic in DefaultRedisCacheWriter.

Split RedisCacheTests.retrieveCacheValueUsingJedis() test logic by test case.

Edit Javadoc.

See spring-projects#2650
Original pull request: spring-projects#2717
jxblum pushed a commit to jxblum/spring-data-redis that referenced this issue Oct 12, 2023
Replace blocking lock wait with non-blocking flow. Add support for asynchronous storage to persist the cache value after retrieval from the value supplier. Introduce AsyncCacheWriter abstraction to improve functional guards.

Reformat code. Remove redundant tests. Revisit deprecation notices with consistent mention of the version in which the deprecation was introduced.

Refine exception messages when RedisCache does not support async retrieval.

See spring-projects#2650
Original pull request: spring-projects#2717
jxblum added a commit to jxblum/spring-data-redis that referenced this issue Oct 12, 2023
Cleanup ambigious, unreadable, unstructured and complex logic in DefaultRedisCacheWriter.

Split RedisCacheTests.retrieveCacheValueUsingJedis() test logic by test case.

Edit Javadoc.

See spring-projects#2650
Original pull request: spring-projects#2717
@jxblum jxblum closed this as completed in 3dac68f Oct 12, 2023
@mp911de mp911de changed the title Adds support for asynchronous retrieval from RedisCache Add support for asynchronous retrieval from RedisCache Oct 13, 2023
@mp911de mp911de added this to the 3.2 RC1 (2023.1.0) milestone Oct 13, 2023
mp911de pushed a commit that referenced this issue Oct 13, 2023
mp911de added a commit that referenced this issue Oct 13, 2023
Replace blocking lock wait with non-blocking flow. Add support for asynchronous storage to persist the cache value after retrieval from the value supplier. Introduce AsyncCacheWriter abstraction to improve functional guards.

Reformat code. Remove redundant tests. Revisit deprecation notices with consistent mention of the version in which the deprecation was introduced.

Refine exception messages when RedisCache does not support async retrieval.

See #2650
Original pull request: #2717
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 a pull request may close this issue.

3 participants