Question

我已经检查了这些问题，但它们并没有帮助我解决我的问题。我使用Spring-data-redis库在我的Spring REST应用程序中使用Redis作为速率限制的键值存储。我测试了巨大的负载。在那里我使用以下代码存储密钥，我也设置过期时间。大多数情况下密钥按预期到期。但有时关键还没到期！

代码段

RedisAtomicInteger counter = counter = new RedisAtomicInteger("mykey");
counter.expire(1, TimeUnit.MINUTES);

我使用redis-cli工具检查了密钥的可用性

键*

和

ttl keyname

redis.conf具有默认值。

有什么建议吗？

编辑1：

完整代码：

该功能位于Aspect

中

public synchronized Object checkLimit(ProceedingJoinPoint joinPoint) throws Exception, Throwable {

        boolean isKeyAvailable = false;
        List<String> keysList = new ArrayList<>();

        Object[] obj = joinPoint.getArgs();
        String randomKey = (String) obj[1];
        int randomLimit = (Integer) obj[2];

        // for RedisTemplate it is already loaded as 

        // @Autowired
        // private RedisTemplate template; 

        // in this class
        Set<String> redisKeys = template.keys(randomKey+"_"randomLimit+"*");
        Iterator<String> it = redisKeys.iterator();
        while (it.hasNext()) {
               String data = it.next();
               keysList.add(data);
        }

        if (keysList.size() > 0) {
            isKeyAvailable = keysList.get(0).contains(randomKey + "_" + randomLimit);
        }

        RedisAtomicInteger counter = null;
        // if the key is not there
        if (!isKeyAvailable) {
              long expiryTimeStamp = 0;
              int timePeriodInMintes = 1;
              expiryTimeStamp = new Date(System.currentTimeMillis() + timePeriodInMintes * 60 * 1000).getTime();
              counter = new RedisAtomicInteger(randomKey+ "_"+ randomLimit + "_" + expiryTimeStamp,template.getConnectionFactory());
              counter.incrementAndGet();
              counter.expire(timePeriodInMintes, TimeUnit.MINUTES);
              break;

        } else {

              String[] keys = keysList.get(0).split("_");
              String rLimit = keys[1];

              counter = new RedisAtomicInteger(keysList.get(0), template.getConnectionFactory());
              int count = counter.get();
              // If count exceeds throw error
              if (count != 0 && count >= Integer.parseInt(rLimit)) {
                    throw new Exception("Error");
               }  
               else {
                    counter.incrementAndGet();
              }
      }
        return joinPoint.proceed();
    }

当这些行运行时

RedisAtomicInteger counter = counter = new RedisAtomicInteger（“mykey”）; counter.expire（1，TimeUnit.MINUTES）;

我可以看到

75672562.380127 [0 10.0.3.133:65462] "KEYS" "mykey_1000*"
75672562.384267 [0 10.0.3.133:65462] "GET" "mykey_1000_1475672621787"
75672562.388856 [0 10.0.3.133:65462] "SET" "mykey_1000_1475672621787" "0"
75672562.391867 [0 10.0.3.133:65462] "INCRBY" "mykey_1000_1475672621787" "1"
75672562.395922 [0 10.0.3.133:65462] "PEXPIRE" "mykey_1000_1475672621787" "60000"
...
75672562.691723 [0 10.0.3.133:65462] "KEYS" "mykey_1000*"
75672562.695562 [0 10.0.3.133:65462] "GET" "mykey_1000_1475672621787"
75672562.695855 [0 10.0.3.133:65462] "GET" "mykey_1000_1475672621787"
75672562.696139 [0 10.0.3.133:65462] "INCRBY" "mykey_1000_1475672621787" "1"

在Redis日志中

，当我在

中“监视”它时

Answer 1

编辑：现在有了更新后的代码，我相信除了您报告的内容之外，您的方法存在根本缺陷。

您实施它的方式需要在生产中运行KEYS - 这很糟糕。当您向外扩展时，您将导致服务器上不断增长的，不必要的系统阻塞负载。正如它上面的每一个文档所说，不在生产中使用keys。请注意，在密钥名称中对到期时间进行编码不会带来任何好处。如果您将密钥名称的那一部分作为创建时间戳，甚至是随机数，则不会改变任何内容。实际上，如果你删除那个位，那么什么都不会改变。

更合理的路线是使用与时间无关的键名。使用适合您的过期句柄。让我们将您的利率限制事项称为＆＃34;会话＆＃34;。您的密钥名称没有时间戳是＆＃34;会话ID＆＃34;。通过设置60秒的到期时间，它将不再在61s标记处可用。因此，您可以安全地增加结果并将结果与您的限制进行比较，而无需知道当前时间或到期时间。您只需要一个静态密钥名称和一个适当的过期设置。

如果INCR是一个不存在的密钥，Redis将返回＆＃34; 1＆＃34;意味着它创建了密钥并在一步/调用中增加了密钥。所以基本上逻辑是这样的：

创建＆＃34;会话＆＃34; ID
使用ID
将结果与限制进行比较
1. if count == 1，将expiration设置为60s
2. id count＆gt;限制，拒绝

步骤3.1很重要。计数为1表示这是Redis中的新密钥，您希望在其上设置到期日期。其他任何意味着应该已经设置过期。如果你在3.2中设置它将打破这个过程，因为它将保留计数器超过60秒。

有了这个，你不需要根据到期时间拥有动态密钥名称，因此不需要使用keys来查明是否存在＆＃34;会话＆＃34;对于速率受限的对象。它还使您的代码更加简单和可预测，并减少到Redis的往返 - 这意味着它将降低Redis的负载并且性能更好。至于如何使用您使用的客户端库，我不能说，因为我不熟悉它。但基本顺序应该是可以翻译的，因为它是相当基本和简单的。

然而，您所展示的是支持过期未发生的断言的任何内容。你所做的只是表明Redis确实被告知并设定了到期日。为了支持您的声明，您需要证明密钥不会过期。这意味着你需要在到期时间之后显示密钥的检索，并且计数器不是＆＃34;重置＆＃34;通过在到期后重新创建。您可以看到过期的一种方法是使用keyspace notifications。有了这个，你就可以看到Redis说钥匙已经过期了。

如果你为了速率限制做了多个窗口，或者如果你有一个更大的窗口（即10分钟），这个过程会失败一点，在这种情况下，排序集可能是一个更合理的选择，以防止前面加载请求 - 如果需要。但是正如你的例子所写的那样，上面的工作就可以了。

为什么Redis密钥没有到期？

1 个答案: