hbase-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From te...@apache.org
Subject svn commit: r1226038 - in /hbase/branches/0.92: CHANGES.txt src/test/java/org/apache/hadoop/hbase/replication/TestReplication.java
Date Sat, 31 Dec 2011 05:02:25 GMT
Author: tedyu
Date: Sat Dec 31 05:02:25 2011
New Revision: 1226038

URL: http://svn.apache.org/viewvc?rev=1226038&view=rev
Log:
HBASE-5112  TestReplication#queueFailover flaky due to potentially
               uninitialized Scan (Jimmy Xiang)

Modified:
    hbase/branches/0.92/CHANGES.txt
    hbase/branches/0.92/src/test/java/org/apache/hadoop/hbase/replication/TestReplication.java

Modified: hbase/branches/0.92/CHANGES.txt
URL: http://svn.apache.org/viewvc/hbase/branches/0.92/CHANGES.txt?rev=1226038&r1=1226037&r2=1226038&view=diff
==============================================================================
--- hbase/branches/0.92/CHANGES.txt (original)
+++ hbase/branches/0.92/CHANGES.txt Sat Dec 31 05:02:25 2011
@@ -525,6 +525,8 @@ Release 0.92.0 - Unreleased
                (Gao Jinchao)
    HBASE-4874  Run tests with non-secure random, some tests hang otherwise (Lars H)
    HBASE-4987  wrong use of incarnation var in SplitLogManager (Prakash Khemani)
+   HBASE-5112  TestReplication#queueFailover flaky due to potentially
+               uninitialized Scan (Jimmy Xiang)
 
   IMPROVEMENTS
    HBASE-3290  Max Compaction Size (Nicolas Spiegelberg via Stack)  

Modified: hbase/branches/0.92/src/test/java/org/apache/hadoop/hbase/replication/TestReplication.java
URL: http://svn.apache.org/viewvc/hbase/branches/0.92/src/test/java/org/apache/hadoop/hbase/replication/TestReplication.java?rev=1226038&r1=1226037&r2=1226038&view=diff
==============================================================================
--- hbase/branches/0.92/src/test/java/org/apache/hadoop/hbase/replication/TestReplication.java
(original)
+++ hbase/branches/0.92/src/test/java/org/apache/hadoop/hbase/replication/TestReplication.java
Sat Dec 31 05:02:25 2011
@@ -656,14 +656,13 @@ public class TestReplication {
       initialCount = res.length;
     }
 
-    Scan scan2 = new Scan();
-
     int lastCount = 0;
 
     for (int i = 0; i < NB_RETRIES; i++) {
       if (i==NB_RETRIES-1) {
         fail("Waited too much time for queueFailover replication");
       }
+      Scan scan2 = new Scan();
       ResultScanner scanner2 = htable2.getScanner(scan2);
       Result[] res2 = scanner2.next(initialCount * 2);
       scanner2.close();
@@ -693,6 +692,7 @@ public class TestReplication {
         }
       }
     };
+    killer.setDaemon(true);
     killer.start();
     return killer;
   }



Mime
View raw message