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

dlmarion pushed a commit to branch 2.1
in repository https://gitbox.apache.org/repos/asf/accumulo.git


The following commit(s) were added to refs/heads/2.1 by this push:
     new 221fb5f120 Fixed intermittent failures in ServiceLockIT.testLockSerial 
(#6514)
221fb5f120 is described below

commit 221fb5f1205c78a243d00991880c0f613734ccee
Author: Dave Marion <[email protected]>
AuthorDate: Mon Aug 24 14:26:25 2026 -0400

    Fixed intermittent failures in ServiceLockIT.testLockSerial (#6514)
    
    Removed one line that was checking that the 2nd lock was not
    holding the lock in ZooKeeper after unlocking the 1st lock. This
    line of code introduced a race condition in the test. Further
    down in the test method we expect that the 2nd lock is holding
    the lock, so that line of code succeeded or failed based on
    timing.
---
 .../main/java/org/apache/accumulo/test/fate/zookeeper/ServiceLockIT.java | 1 -
 1 file changed, 1 deletion(-)

diff --git 
a/test/src/main/java/org/apache/accumulo/test/fate/zookeeper/ServiceLockIT.java 
b/test/src/main/java/org/apache/accumulo/test/fate/zookeeper/ServiceLockIT.java
index ed4efda820..e794666d30 100644
--- 
a/test/src/main/java/org/apache/accumulo/test/fate/zookeeper/ServiceLockIT.java
+++ 
b/test/src/main/java/org/apache/accumulo/test/fate/zookeeper/ServiceLockIT.java
@@ -497,7 +497,6 @@ public class ServiceLockIT {
       zl1.unlock();
       assertFalse(zlw1.isLockHeld());
       assertFalse(zl1.verifyLockAtSource());
-      assertFalse(zl2.verifyLockAtSource());
       zk1.close();
 
       while (!zlw2.isLockHeld()) {

Reply via email to