Commit 528c76d

christophstroblmp911de

authored and

committed

Update RedisCommands.

Add missing commands. Closes: #3191 Original pull request: #3193

1 parent aa29bc5 commit 528c76dCopy full SHA for 528c76d

File tree

2 files changed

+86

-7

lines changed

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

2 files changed

+86

-7

lines changed

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

Lines changed: 85 additions & 6 deletions

Original file line number	Diff line number	Diff line change
`@@ -45,93 +45,146 @@ public enum RedisCommand {`
`45`	`45`	`// -- A`
`46`	`46`	`APPEND("rw", 2, 2), //`
`47`	`47`	`AUTH("rw", 1, 1), //`
	`48`	`+`
`48`	`49`	`// -- B`
`49`	`50`	`BGREWRITEAOF("r", 0, 0, "bgwriteaof"), //`
`50`	`51`	`BGSAVE("r", 0, 0), //`
`51`	`52`	`BITCOUNT("r", 1, 3), //`
	`53`	`+ BITFIELD("rw", 1), //`
	`54`	`+ BITFIELD_RO("r", 1),`
`52`	`55`	`BITOP("rw", 3), //`
`53`	`56`	`BITPOS("r", 2, 4), //`
	`57`	`+ BLMOVE("rw", 4), //`
	`58`	`+ BLMPOP("rw", 4), //`
`54`	`59`	`BLPOP("rw", 2), //`
`55`	`60`	`BRPOP("rw", 2), //`
`56`	`61`	`BRPOPLPUSH("rw", 3), //`
	`62`	`+ BZMPOP("rw", 3), //`
	`63`	`+ BZPOPMAX("rw", 2), //`
	`64`	`+ BZPOPMIN("rw", 2), //`
	`65`	`+`
`57`	`66`	`// -- C`
	`67`	`+ CLIENT_GETREDIR("r", 0, 0), //`
	`68`	`+ CLIENT_ID("r", 0, 0), //`
	`69`	`+ CLIENT_INFO("r", 0, 0), //`
`58`	`70`	`CLIENT_KILL("rw", 1, 1), //`
`59`	`71`	`CLIENT_LIST("r", 0, 0), //`
`60`	`72`	`CLIENT_GETNAME("r", 0, 0), //`
`61`	`73`	`CLIENT_PAUSE("rw", 1, 1), //`
	`74`	`+ CLIENT_SETINFO("w", 1), //`
`62`	`75`	`CLIENT_SETNAME("w", 1, 1), //`
	`76`	`+ CLIENT_NO_EVICT("w", 1, 1, "client no-evict"), //`
	`77`	`+ CLIENT_NO_TOUCH("w", 1, 1, "client no-touch"), //`
	`78`	`+ CLIENT_TRACKING("rw", 1), //`
`63`	`79`	`CONFIG_GET("r", 1, 1, "getconfig"), //`
`64`	`80`	`CONFIG_REWRITE("rw", 0, 0), //`
`65`	`81`	`CONFIG_SET("w", 2, 2, "setconfig"), //`
`66`	`82`	`CONFIG_RESETSTAT("w", 0, 0, "resetconfigstats"), //`
	`83`	`+ COPY("rw", 2), //`
	`84`	`+`
`67`	`85`	`// -- D`
`68`	`86`	`DBSIZE("r", 0, 0), //`
`69`	`87`	`DECR("w", 1, 1), //`
`70`	`88`	`DECRBY("w", 2, 2), //`
`71`	`89`	`DEL("rw", 1), //`
`72`	`90`	`DISCARD("rw", 0, 0), //`
`73`	`91`	`DUMP("r", 1, 1), //`
	`92`	`+`
`74`	`93`	`// -- E`
`75`	`94`	`ECHO("r", 1, 1), //`
`76`	`95`	`EVAL("rw", 2), //`
	`96`	`+ EVAL_RO("r", 2), //`
`77`	`97`	`EVALSHA("rw", 2), //`
	`98`	`+ EVALSHA_RO("r", 2), //`
`78`	`99`	`EXEC("rw", 0, 0), //`
`79`	`100`	`EXISTS("r", 1, 1), //`
`80`	`101`	`EXPIRE("rw", 2), //`
`81`	`102`	`EXPIREAT("rw", 2), //`
	`103`	`+ EXPIRETIME("r", 1), //`
	`104`	`+`
`82`	`105`	`// -- F`
	`106`	`+ FCALL("rw", 2), //`
	`107`	`+ FCALL_RO("r", 2), //`
`83`	`108`	`FLUSHALL("w", 0, 0), //`
`84`	`109`	`FLUSHDB("w", 0, 0), //`
	`110`	`+ FUNCTION_DELETE("w", 1), //`
	`111`	`+ FUNCTION_DUMP("w", 0, 0), //`
	`112`	`+ FUNCTION_FLUSH("w", 0, 0), //`
	`113`	`+ FUNCTION_KILL("w", 0, 0), //`
	`114`	`+`
`85`	`115`	`// -- G`
`86`	`116`	`GET("r", 1, 1), //`
`87`	`117`	`GETBIT("r", 2, 2), //`
	`118`	`+ GETDEL("rw", 1), //`
	`119`	`+ GETEX("rw", 1), //`
`88`	`120`	`GETRANGE("r", 3, 3), //`
`89`	`121`	`GETSET("rw", 2, 2), //`
`90`	`122`	`GEOADD("w", 3), //`
`91`	`123`	`GEODIST("r", 2), //`
`92`	`124`	`GEOHASH("r", 2), //`
`93`	`125`	`GEOPOS("r", 2), //`
`94`		`- GEORADIUS("r", 4), //`
`95`		`- GEORADIUSBYMEMBER("r", 3), //`
	`126`	`+ GEORADIUS("rw", 4), //`
	`127`	`+ GEORADIUS_RO("r", 4), //`
	`128`	`+ GEORADIUSBYMEMBER("rw", 3), //`
	`129`	`+ GEORADIUSBYMEMBER_RO("r", 3), //`
	`130`	`+ GEOSEARCH("r", 1), //`
	`131`	`+ GEOSEARCH_STORE("rw", 1), //`
	`132`	`+`
`96`	`133`	`// -- H`
`97`	`134`	`HDEL("rw", 2), //`
	`135`	`+ HELLO("rw", 0, 0), //`
`98`	`136`	`HEXISTS("r", 2, 2), //`
`99`	`137`	`HGET("r", 2, 2), //`
`100`	`138`	`HGETALL("r", 1, 1), //`
	`139`	`+ HGETDEL("rw", 2), //`
	`140`	`+ HGETEX("rw", 2), //`
`101`	`141`	`HINCRBY("rw", 3, 3), //`
`102`	`142`	`HINCBYFLOAT("rw", 3, 3), //`
`103`	`143`	`HKEYS("r", 1), //`
`104`	`144`	`HLEN("r", 1), //`
`105`	`145`	`HMGET("r", 2), //`
`106`	`146`	`HMSET("w", 3), //`
	`147`	`+ HPOP("rw", 3),`
`107`	`148`	`HSET("w", 3, 3), //`
`108`	`149`	`HSETNX("w", 3, 3), //`
`109`	`150`	`HVALS("r", 1, 1), //`
`110`	`151`	`HEXPIRE("w", 5), //`
`111`	`152`	`HEXPIREAT("w", 5), //`
`112`	`153`	`HPEXPIRE("w", 5), //`
`113`	`154`	`HPEXPIREAT("w", 5), //`
	`155`	`+ HPEXPIRETIME("r", 4), //`
`114`	`156`	`HPERSIST("w", 4), //`
`115`	`157`	`HTTL("r", 4), //`
`116`	`158`	`HPTTL("r", 4), //`
	`159`	`+ HSCAN("r", 2), //`
	`160`	`+ HSTRLEN("r", 2), //`
	`161`	`+`
`117`	`162`	`// -- I`
`118`	`163`	`INCR("rw", 1), //`
	`164`	`+ INCRBY("rw", 2, 2), //`
`119`	`165`	`INCRBYFLOAT("rw", 2, 2), //`
`120`	`166`	`INFO("r", 0), //`
	`167`	`+`
`121`	`168`	`// -- K`
`122`	`169`	`KEYS("r", 1), //`
	`170`	`+`
`123`	`171`	`// -- L`
	`172`	`+ LCS("r", 2), //`
`124`	`173`	`LASTSAVE("r", 0), //`
`125`	`174`	`LINDEX("r", 2, 2), //`
`126`	`175`	`LINSERT("rw", 4, 4), //`
`127`	`176`	`LLEN("r", 1, 1), //`
	`177`	`+ LMOVE("rw", 2), //`
	`178`	`+ LMPOP("rw", 2), //`
`128`	`179`	`LPOP("rw", 1, 1), //`
	`180`	`+ LPOS("r", 2), //`
`129`	`181`	`LPUSH("rw", 2), //`
`130`	`182`	`LPUSHX("rw", 2), //`
`131`	`183`	`LRANGE("r", 3, 3), //`
`132`	`184`	`LREM("rw", 3, 3), //`
`133`	`185`	`LSET("w", 3, 3), //`
`134`	`186`	`LTRIM("w", 3, 3), //`
	`187`	`+`
`135`	`188`	`// -- M`
`136`	`189`	`MGET("r", 1), //`
`137`	`190`	`MIGRATE("rw", 0), //`
`@@ -140,19 +193,26 @@ public enum RedisCommand {`
`140`	`193`	`MSET("w", 2), //`
`141`	`194`	`MSETNX("w", 2), //`
`142`	`195`	`MULTI("rw", 0, 0), //`
	`196`	`+`
`143`	`197`	`// -- P`
`144`	`198`	`PERSIST("rw", 1, 1), //`
`145`	`199`	`PEXPIRE("rw", 2), //`
`146`	`200`	`PEXPIREAT("rw", 2), //`
	`201`	`+ PEXPIRETIME("r", 1), //`
	`202`	`+ PFADD("w", 10), //`
	`203`	`+ PFCOUNT("r", 1), //`
	`204`	`+ PFMERGE("rw", 2), //`
`147`	`205`	`PING("r", 0, 0), //`
`148`	`206`	`PSETEX("w", 3), //`
`149`	`207`	`PSUBSCRIBE("r", 1), //`
`150`	`208`	`PTTL("r", 1, 1), //`
`151`	`209`	`// -- Q`
`152`	`210`	`QUIT("rw", 0, 0), //`
	`211`	`+`
`153`	`212`	`// -- R`
`154`	`213`	`RANDOMKEY("r", 0, 0), //`
`155`		`-`
	`214`	`+ READONLY("w", 0, 0), //`
	`215`	`+ READWRITE("w", 0, 0), //`
`156`	`216`	`RENAME("w", 2, 2), //`
`157`	`217`	`RENAMENX("w", 2, 2), //`
`158`	`218`	`REPLICAOF("w", 2), //`
`@@ -161,9 +221,11 @@ public enum RedisCommand {`
`161`	`221`	`RPOPLPUSH("rw", 2, 2), //`
`162`	`222`	`RPUSH("rw", 2), //`
`163`	`223`	`RPUSHX("rw", 2, 2), //`
	`224`	`+`
`164`	`225`	`// -- S`
`165`	`226`	`SADD("rw", 2), //`
`166`	`227`	`SAVE("rw", 0, 0), //`
	`228`	`+ SCAN("r", 1), //`
`167`	`229`	`SCARD("r", 1, 1), //`
`168`	`230`	`SCRIPT_EXISTS("r", 1), //`
`169`	`231`	`SCRIPT_FLUSH("rw", 0, 0), //`
`@@ -179,28 +241,47 @@ public enum RedisCommand {`
`179`	`241`	`SETRANGE("rw", 3, 3), //`
`180`	`242`	`SHUTDOWN("rw", 0), //`
`181`	`243`	`SINTER("r", 1), //`
	`244`	`+ SINTERCARD("r", 1), //`
`182`	`245`	`SINTERSTORE("rw", 2), //`
`183`	`246`	`SISMEMBER("r", 2), //`
`184`	`247`	`SLAVEOF("w", 2), //`
`185`	`248`	`SLOWLOG("rw", 1), //`
`186`	`249`	`SMEMBERS("r", 1, 1), //`
`187`	`250`	`SMOVE("rw", 3, 3), //`
`188`	`251`	`SORT("rw", 1), //`
	`252`	`+ SORT_RO("r", 1), //`
`189`	`253`	`SPOP("rw", 1, 1), //`
`190`	`254`	`SRANDMEMBER("r", 1, 1), //`
`191`	`255`	`SREM("rw", 2), //`
	`256`	`+ SSCAN("r", 1), //`
`192`	`257`	`STRLEN("r", 1, 1), //`
`193`	`258`	`SUBSCRIBE("rw", 1), //`
	`259`	`+ SUBSTR("r", 3), //`
`194`	`260`	`SUNION("r", 1), //`
`195`	`261`	`SUNIONSTORE("rw ", 2), //`
`196`	`262`	`SYNC("rw", 0, 0), //`
	`263`	`+`
`197`	`264`	`// -- T`
`198`	`265`	`TIME("r", 0, 0), //`
`199`	`266`	`TTL("r", 1, 1), //`
`200`	`267`	`TYPE("r", 1, 1), //`
	`268`	`+`
`201`	`269`	`// -- U`
	`270`	`+ UNLINK("w", 1), //`
`202`	`271`	`UNSUBSCRIBE("rw", 0), //`
`203`	`272`	`UNWATCH("rw", 0, 0), //`
	`273`	`+`
	`274`	`+ // -- V`
	`275`	`+ VADD("w", 3), //`
	`276`	`+ VCARD("r", 1), //`
	`277`	`+ VDIM("r", 1), //`
	`278`	`+ VEMB("r", 2), //`
	`279`	`+ VISMEMBER("r", 2), //`
	`280`	`+ VLINKS("r", 2, 3), //`
	`281`	`+ VRANDMEMBER("r", 1, 2), //`
	`282`	`+ VREM("w", 2), //`
	`283`	`+ VSIM("w", 1), //`
	`284`	`+`
`204`	`285`	`// -- W`
`205`	`286`	`WATCH("rw", 1), //`
`206`	`287`	`// -- Z`
`@@ -220,10 +301,8 @@ public enum RedisCommand {`
`220`	`301`	`ZREVRANK("r", 2, 2), //`
`221`	`302`	`ZSCORE("r", 2, 2), //`
`222`	`303`	`ZUNIONSTORE("rw", 3), //`
`223`		`- SCAN("r", 1), //`
`224`		`- SSCAN("r", 2), //`
`225`		`- HSCAN("r", 2), //`
`226`	`304`	`ZSCAN("r", 2), //`
	`305`	`+`
`227`	`306`	`// -- UNKNOWN / DEFAULT`
`228`	`307`	`UNKNOWN("rw", -1);`
`229`	`308`

`‎src/test/java/org/springframework/data/redis/core/RedisCommandUnitTests.java‎`

Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -135,7 +135,7 @@ void commandRequiresExactNumberOfArgumentsIsCorrect() {`
`135`	`135`
`136`	`136`	`Arrays.stream(RedisCommand.values())`
`137`	`137`	`.forEach(command -> assertThat(command.requiresExactNumberOfArguments())`
`138`		`- .describedAs("Redis command [%s] failed requires exact arguments check").isEqualTo(`
	`138`	`+ .describedAs("Redis command [%s] failed requires exact arguments check".formatted(command.name())).isEqualTo(`
`139`	`139`	`ReflectionTestUtils.getField(command, "minArgs") == ReflectionTestUtils.getField(command, "maxArgs")));`
`140`	`140`	`}`
`141`	`141`

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 528c76d

File tree

2 files changed

2 files changed

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

`‎src/test/java/org/springframework/data/redis/core/RedisCommandUnitTests.java‎`

0 commit comments