ExpireAfterWrite似乎不起作用



我正在尝试使用Google的Guava缓存,我只是以1分钟的间隔使用expireAfterWrite对其进行测试。在我正在处理的应用程序中,有数百个用户。所以我的理解是,当第一个人访问缓存时,我们从数据库中填充它,然后一分钟计时器应该开始倒计时。现在缓存已填满。我们将始终从缓存中获取一分钟的时间段。一分钟后,它将过期,我们再次从数据库中获取它。

但是,似乎每次我在一分钟内访问缓存时,计时器都会重置。为什么?如果我在一分钟内没有访问缓存,缓存就会过期并正常工作。

法典:

 Cache<Long, List<someObj>> cacheMap = CacheBuilder.newBuilder().maximumSize(maxSize).expireAfterWrite(maxTime, TimeUnit.MINUTES).removalListener(new RemovalListener<Long, List<someObj>>() {
    public void onRemoval(RemovalNotification<Long, List<someObj>> notification) {
        if (notification.getCause() == RemovalCause.EXPIRED) {
            logger.info("Value " + notification.getValue() + " has been expired");
        } else {
            logger.info("Just removed for some reason");
        }
    }
}).build();
...
//Convert to ConcurrentHashMap
cacheMap = someCache.asMap();
...

public List<someObj> getAllData(final Long orgId, final User user) {
    // Get requests from cache
    List<Request> cacheList = overdueSlaMap.get(orgId);
    // Fill cached requests if null
    cacheList = fillCacheIfNecessary(orgId, cacheList, overdueSlaMap);  
    return cacheList;
}
//Strategy for reading and writing to cache
//This method is fine. On first load, cache is empty so we fill it from database. Wait one minute, value expires and cache is empty again so we get from cache. 
//However, everytime I refresh the page the one minute timer starts again. Surely, the one minute timer within the Guava Cache should not refresh regardless of how many times I access the cache within that one minute period. 
 private List<someObj> fillCacheIfNecessary(List<someObj> cacheList, ConcurrentMap<Long, List<someObj>> cacheMap) {
        // If the cache is empty then we will try to fill it from the database.
        if (cacheList == null) {
            logger.info("populating cache");
            List<someObj> objList = new ArrayList<someObj>();
            // if bla bla
            if (such and such) {
                objList = service.getFromDatabase(someArg);
            } else {
                objList = service.getFromDatabase(anotherarg);
            }
            // Concurrently lock the new cacheList retrieved from the database.
            List<someObj> newValue = Collections.unmodifiableList(objList);
            // Only insert if new value does not exist in cache map
            List<someObj> oldValue = cacheMap.putIfAbsent(orgId, newValue);
            // If old value already exists then we use the old value otherwise we use the new value
            cacheList = ((oldValue != null && !oldValue.isEmpty()) ? oldValue : newValue);
        }
        return cacheList;
    }

编辑

我正在从控制器调用缓存:

public ModelAndView getNotifications(HttpServletRequest httpServletRequest, HttpServletResponse httpServletResponse) throws Exception {
    User user = getCurrentUser();
    CacheManager cacheManager = new CacheManager();
    List<someObj> objList = new ArrayList<Request>();
    // Get from database
    if (bla bla) {
        objList = getFromDatabase(user);
    }
    // Get from cache
    else {
        Long orgId = user.getUserOrganisation().getId();
        objList = cacheManager.getAllData(orgId, user);      
    }
    return new ModelAndView(SOMEVIEW);
}

它将所需的有用数据传递给方法,该方法将其传递给方法以调用数据库。但是,我现在正在尝试重构我的代码以使用 LoadCache,但这需要我覆盖仅使用一个参数(即键)的 load() 方法。我需要的不仅仅是键,我需要在控制器中调用 cacheManager,因此它会调用适当的方法。什么是加载方法?什么时候使用?何时调用,多久调用一次?它会使我的其他方法无效吗?

我相信

您的问题是您使用从asMap返回的Map作为缓存。 asMap被记录为 Returns a view of the entries stored in this cache as a thread-safe map 。这里没有任何内容表明您可以将Map用作缓存。我建议更新代码以使用 Cache API 并直接在缓存上调用putget(key, Callable)

您可能需要考虑使用LoadingCache,因为我发现 API 更简单。

编辑

因此,首先,仅当缓存中不存在键时,才会调用 load 方法。缓存调用 load 方法来获取应为其键返回的值,然后缓存该值以供下次使用。使用 CacheLoaderCallable 的好处之一是,检索数据的调用是线程安全的,因为即使同时发生对同一键的多次调用,每个键也只调用一次。

如果需要传递更多信息,请按如下方式创建一个包装对象:

 public class MyKey{
       final Long orgId; // true key value
       final User user; // other values needed
       // implement equals and hashCode to ONLY use orgId. This will ensure that
       // the same cached value is returned for all requests with the same
       // orgId which is the ACTUAL key.
  }

您还可以使用常规Cache执行相同的操作,并创建一个包装所需值的Callable

我建议CacheLoader使fillCacheIfNecessary无效.对cache.get()的简单调用将在内部确定该值是否必须检索或已存在于缓存中。它正在为你做get, check if null, retrieve工作。

相关内容

  • 没有找到相关文章

最新更新