• 请不要在回答技术问题时复制粘贴 AI 生成的内容
BBCCBB
V2EX  ›  程序员

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

  •  
  •   BBCCBB · Dec 9, 2020 · 1864 views
    This topic created in 2013 days ago, the information mentioned may be changed or developed.

    代码参见: doUnlock()

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

    1. expire 失败, 不就不会自动过期了?
    2. doUnlcok 失败, 其他的程序如果调用需要加锁的地方不就都卡住了?
    7 replies    2020-12-09 14:08:49 +08:00
    BBCCBB
        1
    BBCCBB  
    OP
       Dec 9, 2020
    😱
    pigmen
        2
    pigmen  
       Dec 9, 2020
    expire 和 set 一起用不就行了,分开的话用 multi exec

    doUnlock 应该是单个 key 的?
    Vedar
        3
    Vedar  
       Dec 9, 2020
    看代码确实有这个问题,而且看 execute 还弄了个 interruptexception 的机制 看起来确实是没拿到锁就会一直等到被打断为止
    BBCCBB
        4
    BBCCBB  
    OP
       Dec 9, 2020
    @pigmen 关键是没找到用 watch multi 的代码..
    BBCCBB
        5
    BBCCBB  
    OP
       Dec 9, 2020
    @Vedar 说是针对这个死锁的问题升级了, 但看不出来有啥升级, 擦.. 难受.
    Vedar
        6
    Vedar  
       Dec 9, 2020
    @BBCCBB 看了一下这段代码的提交记录 最早是 setnx 和 expire 分开的 现在变成 set 增加了 expire 参数。。确实是更加"atomic"了 (笑哭
    BBCCBB
        7
    BBCCBB  
    OP
       Dec 9, 2020
    @Vedar put 方法里的确是 set 里加的 expire

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

    这代码变更太扯了 😅
    About   ·   Help   ·   Advertise   ·   Blog   ·   API   ·   FAQ   ·   Solana   ·   1090 Online   Highest 6679   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 30ms · UTC 23:33 · PVG 07:33 · LAX 16:33 · JFK 19:33
    ♥ Do have faith in what you're doing.