Skip to content

Avoid wrapping RedisConnection into StringRedisConnection via StringRedisTemplate #3193

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 3 commits into from
Closed

Avoid wrapping RedisConnection into StringRedisConnection via StringRedisTemplate #3193

wants to merge 3 commits into from

Conversation

christophstrobl
Copy link
Member

No description provided.

@mp911de mp911de linked an issue Sep 8, 2025 that may be closed by this pull request
SMISMEMBER is a unknown command for ConnectionSplittingInterceptor #3191
Closed
@mp911de mp911de added the type: enhancement A general enhancement label Sep 8, 2025
@mp911de mp911de marked this pull request as ready for review September 8, 2025 08:55
@mp911de mp911de mentioned this pull request Sep 8, 2025
Open
mp911de pushed a commit that referenced this pull request Sep 8, 2025
@christophstrobl @mp911de
Update RedisCommands.
528c76d 
Add missing commands.

Closes: #3191
Original pull request: #3193
mp911de added a commit that referenced this pull request Sep 8, 2025
@mp911de
Polishing.
feeda02 
Use parametrized tests for Command enum verification.

See #3191
Original pull request: #3193
mp911de pushed a commit that referenced this pull request Sep 8, 2025
@christophstrobl @mp911de
Update RedisCommands.
5a687c0 
Add missing commands.

Closes: #3191
Original pull request: #3193
mp911de added a commit that referenced this pull request Sep 8, 2025
@mp911de
Polishing.
62c48d9 
Use parametrized tests for Command enum verification.

See #3191
Original pull request: #3193
mp911de pushed a commit that referenced this pull request Sep 8, 2025
@christophstrobl @mp911de
Update RedisCommands.
cdf473b 
Add missing commands.

Closes: #3191
Original pull request: #3193
mp911de pushed a commit that referenced this pull request Sep 8, 2025
@christophstrobl @mp911de
Remove connection wrapping in StringRedisTemplate.
3a930ce 
Wrapping is not required due to applying proper serializers. Also, the applied StringRedisConnection type cannot be easily consumed from StringRedisTemplate.

See: #3191
Original pull request: #3193
mp911de added a commit that referenced this pull request Sep 8, 2025
@mp911de
Polishing.
5358b19 
Use parametrized tests for Command enum verification.

See #3191
Original pull request: #3193
@mp911de mp911de added this to the 3.4.10 (2024.1.10) milestone Sep 8, 2025
@mp911de mp911de closed this Sep 8, 2025
@mp911de mp911de deleted the issue/3191 branch September 8, 2025 09:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@mp911de mp911de

Labels
type: enhancement A general enhancement
Projects
None yet
Milestone
3.4.10 (2024.1.10)
Development

Successfully merging this pull request may close these issues.

SMISMEMBER is a unknown command for ConnectionSplittingInterceptor
2 participants
@christophstrobl @mp911de