多级缓存下，仅通过环境变量配置本地过期时间无法生效

[Roiocam]

背景

多级缓存下（cacheType = BOTH) 如果不配置注解的 localExpire ，那么通过配置文件指定的本地过期时间不会生效

jetcache:
  local:
    default:
      type: caffeine
      limit: 100
      # 不会生效
      expireAfterWriteInMillis: 100000

核心代码：

Config 只会从注解里拿

jetcache/jetcache-core/src/main/java/com/alicp/jetcache/SimpleCacheManager.java

Line 122 in 984aa2a

boolean useExpireOfSubCache = config.getLocalExpire() != null;

[areyouok]

buildLocal的时候会生效的

[Roiocam]

buildLocal的时候会生效的

LocalCache 这个对象里面的值会生效，但是 MultiLevelCache 的逻辑不会，取的是 MultiLevelCache 里面的过期时间

jetcache/jetcache-core/src/main/java/com/alicp/jetcache/MultiLevelCache.java

Lines 64 to 67 in 984aa2a

	if (config.isUseExpireOfSubCache()) {
	return PUT(key, value, 0, null);
	} else {
	return PUT(key, value, config().getExpireAfterWriteInMillis(), TimeUnit.MILLISECONDS);

而这个 MultiLevelCache 过期时间的取值逻辑是最后一个 Cache 的过期时间（也就是远程过期时间）：

jetcache/jetcache-core/src/main/java/com/alicp/jetcache/MultiLevelCache.java

Lines 28 to 31 in 984aa2a

	CacheConfig lastConfig = caches[caches.length - 1].config();
	config = new MultiLevelCacheConfig<>();
	config.setCaches(Arrays.asList(caches));
	config.setExpireAfterWriteInMillis(lastConfig.getExpireAfterWriteInMillis());

jetcache/jetcache-core/src/main/java/com/alicp/jetcache/SimpleCacheManager.java

Lines 123 to 126 in 984aa2a

	cache = MultiLevelCacheBuilder.createMultiLevelCacheBuilder()
	.expireAfterWrite(remote.config().getExpireAfterWriteInMillis(), TimeUnit.MILLISECONDS)
	.addCache(local, remote)
	.useExpireOfSubCache(useExpireOfSubCache)

[areyouok]

我明白你的意思了，你的意思是useExpireOfSubCache是多余的。

但当时就这么设计的，现在不确定是否应该改。

[Roiocam]

我明白你的意思了，你的意思是useExpireOfSubCache是多余的。

但当时就这么设计的，现在不确定是否应该改。

嗯，有点像是，如果 LocalCache 和 RemoteCache 各自能通过环境变量覆盖，那么 MultiLevelCache 应该不需要处理这一部分逻辑。现在的代码导致做不到：

环境变量 -> 应用配置 -> 硬编码

这种优先级了

[areyouok]

localExpire是18年后来加的，原先有两层的话，就以remote的为准

[Roiocam]

localExpire是18年后来加的，原先有两层的话，就以remote的为准

我提了个 PR #916 请您看一下？去掉了 useExpireOfSubCache：

• 总是让每层 Caches 选择自己参数的过期时间，除非是用户自己指定的覆盖
• 对于 GET 本地过期，去拿远程后，会用远程的过期时间更新本地，但又尊重本地缓存的过期时间

[areyouok]

我还没细看，但至少有两个问题：

1. 需要保证兼容性。已经公开的api是不能删除的，不用了可以标注为废弃。
2. 不要有中文注释。

[Roiocam]

我还没细看，但至少有两个问题：

1. 需要保证兼容性。已经公开的api是不能删除的，不用了可以标注为废弃。
2. 不要有中文注释。

ok, 修复了