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

最新推荐文章于 2024-04-25 11:38:45 发布

lanicc

最新推荐文章于 2024-04-25 11:38:45 发布

阅读量2.2k

点赞数 5

文章标签： java spring 数据库

本文链接：https://blog.csdn.net/qq_26824159/article/details/126116562

版权

发现问题

看代码时，突然注意到了 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. 更改的
在这里插入图片描述