You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@curator.apache.org by ti...@apache.org on 2023/08/13 15:49:55 UTC

[curator] branch master updated: CURATOR-681: Fix flaky TestInterProcessMutex.testReentrantSingleLock (#470)

This is an automated email from the ASF dual-hosted git repository.

tison pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/curator.git


The following commit(s) were added to refs/heads/master by this push:
     new 260afcda CURATOR-681: Fix flaky TestInterProcessMutex.testReentrantSingleLock (#470)
260afcda is described below

commit 260afcdadbadde87ce7a8c2a904afdc3c1013884
Author: Kezhu Wang <ke...@apache.org>
AuthorDate: Sun Aug 13 23:49:49 2023 +0800

    CURATOR-681: Fix flaky TestInterProcessMutex.testReentrantSingleLock (#470)
---
 .../curator/framework/recipes/locks/TestInterProcessMutexBase.java      | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/curator-recipes/src/test/java/org/apache/curator/framework/recipes/locks/TestInterProcessMutexBase.java b/curator-recipes/src/test/java/org/apache/curator/framework/recipes/locks/TestInterProcessMutexBase.java
index 92cf397f..4ea7c7b9 100644
--- a/curator-recipes/src/test/java/org/apache/curator/framework/recipes/locks/TestInterProcessMutexBase.java
+++ b/curator-recipes/src/test/java/org/apache/curator/framework/recipes/locks/TestInterProcessMutexBase.java
@@ -283,8 +283,8 @@ public abstract class TestInterProcessMutexBase extends BaseClassForTests {
                                 Thread.sleep(100);
                             }
                         } finally {
-                            mutex.release();
                             hasLock.set(false);
+                            mutex.release();
                         }
                         return null;
                     }