You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by sy...@apache.org on 2015/10/24 00:49:31 UTC

[11/50] [abbrv] hbase git commit: HBASE-14647 Disable TestWALProcedureStoreOnHDFS#testWalRollOnLowReplication

HBASE-14647 Disable TestWALProcedureStoreOnHDFS#testWalRollOnLowReplication


Project: http://git-wip-us.apache.org/repos/asf/hbase/repo
Commit: http://git-wip-us.apache.org/repos/asf/hbase/commit/c1f04420
Tree: http://git-wip-us.apache.org/repos/asf/hbase/tree/c1f04420
Diff: http://git-wip-us.apache.org/repos/asf/hbase/diff/c1f04420

Branch: refs/heads/hbase-12439
Commit: c1f0442045f44fcbb3935f9244794929a5d0caea
Parents: ea0cf39
Author: stack <st...@apache.org>
Authored: Mon Oct 19 10:39:09 2015 -0700
Committer: stack <st...@apache.org>
Committed: Mon Oct 19 10:39:09 2015 -0700

----------------------------------------------------------------------
 .../master/procedure/TestWALProcedureStoreOnHDFS.java  | 13 +++++++++----
 1 file changed, 9 insertions(+), 4 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/hbase/blob/c1f04420/hbase-server/src/test/java/org/apache/hadoop/hbase/master/procedure/TestWALProcedureStoreOnHDFS.java
----------------------------------------------------------------------
diff --git a/hbase-server/src/test/java/org/apache/hadoop/hbase/master/procedure/TestWALProcedureStoreOnHDFS.java b/hbase-server/src/test/java/org/apache/hadoop/hbase/master/procedure/TestWALProcedureStoreOnHDFS.java
index 8712c80..f615c1a 100644
--- a/hbase-server/src/test/java/org/apache/hadoop/hbase/master/procedure/TestWALProcedureStoreOnHDFS.java
+++ b/hbase-server/src/test/java/org/apache/hadoop/hbase/master/procedure/TestWALProcedureStoreOnHDFS.java
@@ -55,6 +55,7 @@ import org.apache.hadoop.hdfs.server.datanode.DataNode;
 
 import org.junit.After;
 import org.junit.Before;
+import org.junit.Ignore;
 import org.junit.Test;
 import org.junit.experimental.categories.Category;
 import org.mockito.Mockito;
@@ -182,7 +183,7 @@ public class TestWALProcedureStoreOnHDFS {
                                    reCount.get() < thread.length);
   }
 
-  @Test(timeout=60000)
+  @Ignore ("Needs work") @Test(timeout=60000)
   public void testWalRollOnLowReplication() throws Exception {
     store.unregisterListener(stopProcedureListener);
     store.registerListener(new ProcedureStore.ProcedureStoreListener() {
@@ -191,6 +192,7 @@ public class TestWALProcedureStoreOnHDFS {
 
       @Override
       public void abortProcess() {
+        LOG.info("Aborted!!!!");
       }
     });
     int dnCount = 0;
@@ -203,9 +205,12 @@ public class TestWALProcedureStoreOnHDFS {
         String msg = re.getMessage();
         // We could get a sync failed here...if the test cluster is crawling such that DN recovery
         // is taking a long time. If we've done enough passes, just finish up the test as a 'pass'
-        if (msg != null && msg.toLowerCase().contains("sync aborted") && i > 50) {
-          LOG.info("Returning early... We ran enough of this test", re);
-          return;
+        if (msg != null && msg.toLowerCase().contains("sync aborted")) {
+          LOG.info("i=" + i, re);
+          if (i > 50) {
+            LOG.info("Returning early... i=" + i + "...We ran enough of this test", re);
+            return;
+          }
         }
         throw re;
       }