RedisTemplate的线程安全问题&水了一堆pr

发现问题

看代码时，突然注意到了 RedisTemplate 中 OpsFor** 的代码，例如下面的 opsForValue 方法实现

	@Override
	public ValueOperations<K, V> opsForValue() {

		if (valueOps == null) {
			valueOps = new DefaultValueOperations<>(this);
		}
		return valueOps;
	}
1
2
3
4
5
6
7
8

当时很疑惑，这么写，那 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;
	}
1
2
3
4
5
6
7
8
9
10
11
12

这种 double-check 的惰性初始化才能保证线程安全呀。

寻求答案

于是 github 上搜了一下有没有人反馈这个问题，还真有

1. 第一个是说 opsForHash 方法中的 DefaultHashOperations 对象没有缓存，而是每次都创建一个新的；
2. 第二个和我上面的问题是一致的，他也提交了 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

真真的意外收获