Commit c87d772

committed

Add missing RedisCommand enum entries for ZSet WithScores methods.

The rangeWithScores family of methods (zRangeWithScores, zRangeByScoreWithScores, zRevRangeWithScores, zRevRangeByScoreWithScores) were missing from the RedisCommand enumeration, causing ConnectionSplittingInterceptor to resolve them as UNKNOWN commands. This resulted in read-only methods being incorrectly queued in MULTI/EXEC blocks instead of executing immediately on separate connections, causing them to return null in transactional contexts. This commit adds the ZRANGEWITHSCORES, ZRANGEBYSCOREWITHSCORES, ZREVRANGEWITHSCORES, ZREVRANGEBYSCOREWITHSCORES enum entries to RedisCommand.java. The fix ensures these read-only methods are properly recognized and execute immediately in transactional contexts. Original Pull Request: #3225 Fixes: #3187 Signed-off-by: LeeHyungGeol <rjf1138@naver.com> (cherry picked from commit 96f250d) Signed-off-by: Chris Bono <chris.bono@broadcom.com>

1 parent d136ea3 commit c87d772Copy full SHA for c87d772

File tree

1 file changed

-0

lines changed

src/main/java/org/springframework/data/redis/core
- RedisCommand.java

1 file changed

-0

lines changed

`‎src/main/java/org/springframework/data/redis/core/RedisCommand.java‎`

Lines changed: 5 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -35,6 +35,7 @@`
`35`	`35`	`* @author Oscar Cai`
`36`	`36`	`* @author Sébastien Volle`
`37`	`37`	`* @author John Blum`
	`38`	`+ * @author LeeHyungGeol`
`38`	`39`	`* @since 1.3`
`39`	`40`	`* @link <a href=`
`40`	`41`	`* "https://github.com/antirez/redis/blob/843de8b786562d8d77c78d83a971060adc61f77a/src/server.c#L180">Redis`
`@@ -292,12 +293,16 @@ public enum RedisCommand {`
`292`	`293`	`ZINTERSTORE("rw", 3), //`
`293`	`294`	`ZRANGE("r", 3), //`
`294`	`295`	`ZRANGEBYSCORE("r", 3), //`
	`296`	`+ ZRANGEWITHSCORES("r", 3), //`
	`297`	`+ ZRANGEBYSCOREWITHSCORES("r", 2), //`
`295`	`298`	`ZRANK("r", 2, 2), //`
`296`	`299`	`ZREM("rw", 2), //`
`297`	`300`	`ZREMRANGEBYRANK("rw", 3, 3), //`
`298`	`301`	`ZREMRANGEBYSCORE("rw", 3, 3), //`
`299`	`302`	`ZREVRANGE("r", 3), //`
`300`	`303`	`ZREVRANGEBYSCORE("r", 3), //`
	`304`	`+ ZREVRANGEWITHSCORES("r", 3), //`
	`305`	`+ ZREVRANGEBYSCOREWITHSCORES("r", 2), //`
`301`	`306`	`ZREVRANK("r", 2, 2), //`
`302`	`307`	`ZSCORE("r", 2, 2), //`
`303`	`308`	`ZUNIONSTORE("rw", 3), //`

0 commit comments

Comments

(0)

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Commit c87d772

File tree

1 file changed

1 file changed

`‎src/main/java/org/springframework/data/redis/core/RedisCommand.java‎`

0 commit comments