[Spring data redis] RedisCacheWriter.lockingRedisCacheWriter#putIfAbsent 不是原子性操作

V2EX = way to explore

V2EX 是一个关于分享和探索的地方

现在注册

已注册用户请登录

这是一个创建于 1445 天前的主题，其中的信息可能已经有所发展或是发生改变。

代码参见: doUnlock()

有清楚这一块逻辑的老哥吗? 我看了很多引用的地方和这一块的代码, 怎么看都是这几步操作不是原子性的, 随便一个操作出问题不就挂了.. 比如

7 条回复 • 2020-12-09 14:08:49 +08:00

BBCCBB

2020-12-09 10:45:54 +08:00

😱

pigmen

2020-12-09 10:58:10 +08:00

expire 和 set 一起用不就行了，分开的话用 multi exec

doUnlock 应该是单个 key 的？

Vedar

2020-12-09 11:10:42 +08:00

看代码确实有这个问题，而且看 execute 还弄了个 interruptexception 的机制看起来确实是没拿到锁就会一直等到被打断为止

BBCCBB

2020-12-09 12:30:33 +08:00

@pigmen 关键是没找到用 watch multi 的代码..

BBCCBB

2020-12-09 12:31:36 +08:00

@Vedar 说是针对这个死锁的问题升级了, 但看不出来有啥升级, 擦.. 难受.

Vedar

2020-12-09 13:52:40 +08:00

@BBCCBB 看了一下这段代码的提交记录最早是 setnx 和 expire 分开的现在变成 set 增加了 expire 参数。。确实是更加"atomic"了（笑哭

BBCCBB

2020-12-09 14:08:49 +08:00

@Vedar put 方法里的确是 set 里加的 expire

但 putIfAbsent 这个方法里依然是 lock -> setnx -> expire -> unlock 这个流程,

这代码变更太扯了 😅