Redisson distributed lock unlocking exception

Problem phenomenon

In program redission perform unlock() An error is as follows ：

java.lang.IllegalMonitorStateException: attempt to unlock lock, not locked by current thread by node id:

According to the error information ： Try unlocking , Instead of the current thread by node ID lock

Problem recurrence

// Get lock object 
RLock lock = redissonClient.getLock(key);

try{
   // Get the lock 
   boolean tryLock = lock.tryLock(5, TimeUnit.SECONDS);
   if (!tryLock) {
   // Throw business exception 
   }
}catch(){
    // Capture exception 
}finally{
    // Unlock 
    lock.unlock();
}

The screening process

Code above , Whether or not a thread acquires a lock , It is necessary to implement the unlocking method , When a thread does not acquire a lock , perform unlock() Will report java.lang.IllegalMonitorStateException: attempt to unlock lock, not locked by current thread by node id: error

Solution

1、 Increase judgment when unlocking

// Determine the to unlock key Whether it has been locked ; Determine the to unlock key Whether it is held by the current thread 
if (lock.isLocked() && lock.isHeldByCurrentThread()) {
    lock.unlock();
}

2、 Optimize the code

In execution unlock() Make sure that the thread has acquired the lock

// Get lock object 
RLock lock = redissonClient.getLock(key);
boolean tryLock;
try {
    tryLock = lock.tryLock(5, TimeUnit.SECONDS);
} catch (InterruptedException e) {
    throw new Exception(" Failed to acquire a distributed lock , Please try again later ");
}

if (!tryLock) {
    throw new Exception(" Please try again later ");
}

try{
   // Throw business exception 
}catch(){
    // Capture exception 
}finally{
    // Unlock 
    lock.unlock();
}