fix: when getting the lock by blocking methods, such as waiting in the blocking queue, lock() must be put outside the try block.

Author: Ahlfors

chaosblade-exec-plugin/chaosblade-exec-plugin-jvm/src/main/java/com/alibaba/chaosblade/exec/plugin/jvm/script/base/cache/AbstractScriptCache.java

@@ -40,8 +40,8 @@ public AbstractScriptCache(int cacheSize) {
     public void put(K k, V v) {
         ObjectsUtil.requireNonNull(v);
         Lock lock = readWriteLock.writeLock();
+        lock.lock();
         try {
-            lock.lock();
             cacheMap.put(k, v);
         } finally {
             lock.unlock();
@@ -51,8 +51,8 @@ public void put(K k, V v) {
     @Override
     public V get(K k) {
         Lock lock = readWriteLock.readLock();
+        lock.lock();
         try {
-            lock.lock();
             return cacheMap.get(k);
         } finally {
             lock.unlock();
@@ -63,8 +63,8 @@ public V get(K k) {
     public boolean evict(K k) {
         if (cacheMap.containsKey(k)) {
             Lock lock = readWriteLock.writeLock();
+            lock.lock();
             try {
-                lock.lock();
                 cacheMap.remove(k);
                 return true;
             } finally {
@@ -77,8 +77,8 @@ public boolean evict(K k) {
     @Override
     public void clean() {
         Lock lock = readWriteLock.writeLock();
+        lock.lock();
         try {
-            lock.lock();
             cacheMap.clear();
         } finally {
             lock.unlock();