Add configurable deletion strategy for Redis repository operations #2294 #3162
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
This PR implements configurable deletion strategy for Redis repository operations, allowing applications to choose between synchronous
DEL
and asynchronousUNLINK
commands for better performance optimization.Background
Currently, Spring Data Redis repositories always use the blocking
DEL
command when updating existing keys, which can cause performance issues in update-intensive applications, especially when dealing with large data structures. Redis 4.0+ introduced the non-blockingUNLINK
command as an alternative.Changes
Core Implementation
DeletionStrategy
enum: DefinesDEL
(default) andUNLINK
optionsRedisKeyValueAdapter
:setDeletionStrategy()
andgetDeletionStrategy()
methodsput()
,delete()
, and expiration handling to use configured strategyapplyDeletionStrategy()
utility method@EnableRedisRepositories
: Extended withdeletionStrategy
attributeRedisRepositoryConfigurationExtension
: Updated to propagate configurationTest Coverage
RedisKeyValueAdapterTests
: Basic functionality and configuration testsRedisRepositoryConfigurationExtensionUnitTests
: Annotation configuration testsCloses #2294