Skip to content

Commit

Permalink
Merge pull request #167 from conductor-oss/concrrent_exec_limit
Browse files Browse the repository at this point in the history
Fix for concurrentExecLimit
  • Loading branch information
v1r3n committed May 26, 2024
2 parents 31e2024 + 1f2988f commit 68e3059
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -289,7 +289,7 @@ public boolean exceedsLimit(TaskModel task) {
jedisProxy.zaddnx(rateLimitKey, score, taskId);
recordRedisDaoRequests("checkTaskRateLimiting", task.getTaskType(), task.getWorkflowType());

Set<String> ids = jedisProxy.zrangeByScore(rateLimitKey, 0, score + 1, limit);
Set<String> ids = jedisProxy.zrangeByScore(rateLimitKey, 0, score + 1, Integer.MAX_VALUE);
boolean rateLimited = !ids.contains(taskId);
if (rateLimited) {
LOGGER.info(
Expand Down

0 comments on commit 68e3059

Please sign in to comment.