zookeeper-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From iv...@apache.org
Subject svn commit: r1430041 - in /zookeeper/bookkeeper/trunk: CHANGES.txt bookkeeper-server/src/test/java/org/apache/bookkeeper/replication/AuditorBookieTest.java
Date Mon, 07 Jan 2013 21:53:54 GMT
Author: ivank
Date: Mon Jan  7 21:53:54 2013
New Revision: 1430041

URL: http://svn.apache.org/viewvc?rev=1430041&view=rev
Log:
BOOKKEEPER-534: Flakeyness in AuditorBookieTest (umamahesh via ivank)

Modified:
    zookeeper/bookkeeper/trunk/CHANGES.txt
    zookeeper/bookkeeper/trunk/bookkeeper-server/src/test/java/org/apache/bookkeeper/replication/AuditorBookieTest.java

Modified: zookeeper/bookkeeper/trunk/CHANGES.txt
URL: http://svn.apache.org/viewvc/zookeeper/bookkeeper/trunk/CHANGES.txt?rev=1430041&r1=1430040&r2=1430041&view=diff
==============================================================================
--- zookeeper/bookkeeper/trunk/CHANGES.txt (original)
+++ zookeeper/bookkeeper/trunk/CHANGES.txt Mon Jan  7 21:53:54 2013
@@ -162,6 +162,8 @@ Trunk (unreleased changes)
 
         BOOKKEEPER-355: Ledger recovery will mark ledger as closed with -1, in case of slow
bookie is added to ensemble during recovery add (ivank)
 
+        BOOKKEEPER-534: Flakeyness in AuditorBookieTest (umamahesh via ivank)
+
       hedwig-protocol:
 
         BOOKKEEPER-394: CompositeException message is not useful (Stu Hood via sijie)

Modified: zookeeper/bookkeeper/trunk/bookkeeper-server/src/test/java/org/apache/bookkeeper/replication/AuditorBookieTest.java
URL: http://svn.apache.org/viewvc/zookeeper/bookkeeper/trunk/bookkeeper-server/src/test/java/org/apache/bookkeeper/replication/AuditorBookieTest.java?rev=1430041&r1=1430040&r2=1430041&view=diff
==============================================================================
--- zookeeper/bookkeeper/trunk/bookkeeper-server/src/test/java/org/apache/bookkeeper/replication/AuditorBookieTest.java
(original)
+++ zookeeper/bookkeeper/trunk/bookkeeper-server/src/test/java/org/apache/bookkeeper/replication/AuditorBookieTest.java
Mon Jan  7 21:53:54 2013
@@ -293,12 +293,11 @@ public class AuditorBookieTest extends B
             List<BookieServer> auditors = getAuditorBookie();
             if (auditors.size() > 0) {
                 newAuditor = auditors.get(0);
-                if (auditor == newAuditor) {
-                    Thread.sleep(500);
-                } else {
+                if (auditor != newAuditor) {
                     break;
                 }
             }
+            Thread.sleep(500);
             retryCount--;
         }
         Assert.assertNotNull(



Mime
View raw message