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

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


The following commit(s) were added to refs/heads/master by this push:
     new cb84b27  HBASE-22234 Fix flaky 
TestHbck#testRecoverSplitAfterMetaUpdated
cb84b27 is described below

commit cb84b2788aeed6aad2bc4b1acb98290e060861e2
Author: tianjingyun <[email protected]>
AuthorDate: Sun Apr 14 21:59:59 2019 +0800

    HBASE-22234 Fix flaky TestHbck#testRecoverSplitAfterMetaUpdated
---
 .../src/test/java/org/apache/hadoop/hbase/client/TestHbck.java      | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git 
a/hbase-server/src/test/java/org/apache/hadoop/hbase/client/TestHbck.java 
b/hbase-server/src/test/java/org/apache/hadoop/hbase/client/TestHbck.java
index 8ee00b8..d9a7ca9 100644
--- a/hbase-server/src/test/java/org/apache/hadoop/hbase/client/TestHbck.java
+++ b/hbase-server/src/test/java/org/apache/hadoop/hbase/client/TestHbck.java
@@ -253,7 +253,7 @@ public class TestHbck {
       Assert.assertEquals(1, result.size());
       hbck.assigns(Arrays.asList(result.keySet().toArray(new 
String[0])).stream()
           .map(regionName -> 
regionName.split("\\.")[1]).collect(Collectors.toList()));
-      
ProcedureTestingUtility.waitNoProcedureRunning(master.getMasterProcedureExecutor());
+      
ProcedureTestingUtility.waitAllProcedures(master.getMasterProcedureExecutor());
       // now the state should be fixed
       result = 
hbck.getFailedSplitMergeLegacyRegions(Arrays.asList(TableName.valueOf(testTable)));
       Assert.assertEquals(0, result.size());
@@ -290,7 +290,7 @@ public class TestHbck {
       Assert.assertEquals(2, result.size());
       hbck.assigns(Arrays.asList(result.keySet().toArray(new 
String[0])).stream()
           .map(regionName -> 
regionName.split("\\.")[1]).collect(Collectors.toList()));
-      
ProcedureTestingUtility.waitNoProcedureRunning(master.getMasterProcedureExecutor());
+      
ProcedureTestingUtility.waitAllProcedures(master.getMasterProcedureExecutor());
       // now the state should be fixed
       result = 
hbck.getFailedSplitMergeLegacyRegions(Arrays.asList(TableName.valueOf(testTable)));
       Assert.assertEquals(0, result.size());
@@ -310,7 +310,7 @@ public class TestHbck {
       Assert.assertEquals(2, result.size());
       hbck.assigns(Arrays.asList(result.keySet().toArray(new 
String[0])).stream()
           .map(regionName -> 
regionName.split("\\.")[1]).collect(Collectors.toList()));
-      
ProcedureTestingUtility.waitNoProcedureRunning(master.getMasterProcedureExecutor());
+      
ProcedureTestingUtility.waitAllProcedures(master.getMasterProcedureExecutor());
       // now the state should be fixed
       result = 
hbck.getFailedSplitMergeLegacyRegions(Arrays.asList(TableName.valueOf(testTable)));
       Assert.assertEquals(0, result.size());

Reply via email to