Fixed - RRateLimiter tryAcquire() and availablePermits() method throw too many results to unpack error #3526

4 years ago · 8563ddb3d4
parent 9bd3d88957
commit 8563ddb3d4
1 changed files with 2 additions and 2 deletions
--- a/redisson/src/main/java/org/redisson/RedissonRateLimiter.java
+++ b/redisson/src/main/java/org/redisson/RedissonRateLimiter.java
@ -205,7 +205,7 @@ public class RedissonRateLimiter extends RedissonExpirable implements RRateLimit
                     + "end; "

                     + "if released > 0 then "
-                          + "redis.call('zrem', permitsName, unpack(expiredValues)); "
+                          + "redis.call('zremrangebyscore', permitsName, 0, tonumber(ARGV[2]) - interval); "
                          + "currentValue = tonumber(currentValue) + released; "
                          + "redis.call('set', valueName, currentValue);"
                     + "end;"
@ -320,7 +320,7 @@ public class RedissonRateLimiter extends RedissonExpirable implements RRateLimit
                     + "end; "

                     + "if released > 0 then "
-                          + "redis.call('zrem', permitsName, unpack(expiredValues)); "
+                          + "redis.call('zremrangebyscore', permitsName, 0, tonumber(ARGV[1]) - interval); "
                          + "currentValue = tonumber(currentValue) + released; "
                          + "redis.call('set', valueName, currentValue);"
                     + "end;"