发现问题
看代码时,突然注意到了 RedisTemplate 中 OpsFor** 的代码,例如下面的 opsForValue 方法实现
@Override
public ValueOperations<K, V> opsForValue() {
if (valueOps == null) {
valueOps = new DefaultValueOperations<>(this);
}
return valueOps;
}
当时很疑惑,这么写,那 RedisTemplate 到底是线程安全的,还是非线程安全的呢?看一下它的 Java doc
从 java doc 上的描述来看,设计者是想将此类设计为一个线程安全的类。
但是,这个似乎和 opsFor** 这些方法的实现有些出入,例如上面的 opsForValue 方法,valueOps 应该是想惰性初始化,但是没有进行 double-check,多线程环境下就会有问题啊,我想这段代码应该是可以优化的,例如修改为下面这种
@Override
public ValueOperations<K, V> opsForValue() {
if (valueOps == null) {
synchronized (this) {
if (valueOps == null) {
valueOps = new DefaultValueOperations<>(this);
}
}
}
return valueOps;
}
这种 double-check 的惰性初始化才能保证线程安全呀。
寻求答案
于是 github 上搜了一下有没有人反馈这个问题,还真有
- 第一个是说 opsForHash 方法中的 DefaultHashOperations 对象没有缓存,而是每次都创建一个新的;
- 第二个和我上面的问题是一致的,他也提交了 PR
PR 地址:https://github.com/spring-projects/spring-data-redis/pull/479
修复的思路不是通过 double-check + 惰性初始化,而是通过提前初始化 + final 修饰为不可变的变量,可以看下面的更改内容对比
意外收获
在看大佬提交的 PR 时,我尝试在本地构建源码,按照文档操作
很简单,就一行命令,可就在这时,出现了一个错误
即,当我使用 java8 编译源码的时候,提示我要用 17!!!
那文档上描述的岂不是错误的,哈哈哈,一个水 pr 的好机会。
但为了严谨,要找到为什么需要 17
于是,找啊找,发现这个是在 spring-data-parent
中的一个插件定义的限制,spring-data-parent
是 spring-data-redis
的 parent。
spring-data-parent 的项目地址是 https://github.com/spring-projects/spring-data-build,这个项目的作用是
This repository contains common infrastructure to be used by Spring Data modules that build with Maven.
其他的 spring-data-xx 项目都会用 spring-data-parent 作为项目依赖的 parent,而我发现 spring-data-redis 中依赖的是 spring-data-parent 中最新的 snapshot 版本,也就是只要 spring-data-parent 的 snapshot 变了,spring-data-redis 也会受影响。
Spring-data-parent 需要 java17 的限制是在这个提交中Increment Java version in enforcer plugin, update to latest Artifactory plugin version. 更改的
spring-data-redis 受到了影响,那其他的 spring-data 模块应该也会这样,我又查看了其他的模块,有
- spring-data-elasticsearch
- Spring-data-jpa
- …
有些我都没有用过,但是现在他们的文档描述不对了,需要更新了
于是我给每个仓库都提了 pr,哈哈哈。elasticsearch 的已经被大佬们合进去了。
Fix the JDK version required to build from the source code
真真的意外收获