Apache
/
hbase
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Revert "HBASE-20194 Remove the explicit timeout config for TestTruncateTableProcedure" 

This reverts commit cc6fc1f04f.
This commit is contained in:
Guanghao Zhang 2019-03-04 15:32:13 +08:00
parent ab8d9e736f
commit f7aca3fa06
1 changed files with 20 additions and 18 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

32
hbase-server/src/test/java/org/apache/hadoop/hbase/master/procedure/TestTruncateTableProcedure.java
View File

@ -55,7 +55,7 @@ import org.slf4j.Logger;
import org.slf4j.LoggerFactory; import org.slf4j.LoggerFactory;
import org.apache.hadoop.hbase.shaded.protobuf.generated.MasterProcedureProtos; import org.apache.hadoop.hbase.shaded.protobuf.generated.MasterProcedureProtos;
@Category({ MasterTests.class, MediumTests.class }) @Category({MasterTests.class, MediumTests.class})
public class TestTruncateTableProcedure extends TestTableDDLProcedureBase { public class TestTruncateTableProcedure extends TestTableDDLProcedureBase {
@ClassRule @ClassRule
@ -316,7 +316,7 @@ public class TestTruncateTableProcedure extends TestTableDDLProcedureBase {
ProcedureTestingUtility.assertProcNotFailed(procExec, procId); ProcedureTestingUtility.assertProcNotFailed(procExec, procId);
} }
@Test @Test(timeout = 60000)
public void testTruncateWithPreserveAfterSplit() throws Exception { public void testTruncateWithPreserveAfterSplit() throws Exception {
String[] families = new String[] { "f1", "f2" }; String[] families = new String[] { "f1", "f2" };
byte[][] splitKeys = byte[][] splitKeys =
@ -324,10 +324,10 @@ public class TestTruncateTableProcedure extends TestTableDDLProcedureBase {
TableName tableName = TableName.valueOf(name.getMethodName()); TableName tableName = TableName.valueOf(name.getMethodName());
RegionInfo[] regions = MasterProcedureTestingUtility.createTable(getMasterProcedureExecutor(), RegionInfo[] regions = MasterProcedureTestingUtility.createTable(getMasterProcedureExecutor(),
tableName, splitKeys, families); tableName, splitKeys, families);
splitAndTruncate(tableName, regions, 1); splitAndTruncate(tableName, regions);
} }
@Test @Test(timeout = 60000)
public void testTruncatePreserveWithReplicaRegionAfterSplit() throws Exception { public void testTruncatePreserveWithReplicaRegionAfterSplit() throws Exception {
String[] families = new String[] { "f1", "f2" }; String[] families = new String[] { "f1", "f2" };
byte[][] splitKeys = byte[][] splitKeys =
@ -335,8 +335,10 @@ public class TestTruncateTableProcedure extends TestTableDDLProcedureBase {
TableName tableName = TableName.valueOf(name.getMethodName()); TableName tableName = TableName.valueOf(name.getMethodName());
// create a table with region replications // create a table with region replications
TableDescriptor htd = TableDescriptorBuilder.newBuilder(tableName).setRegionReplication(3) TableDescriptor htd = TableDescriptorBuilder.newBuilder(tableName)
.setColumnFamilies(Arrays.stream(families) .setRegionReplication(3)
.setColumnFamilies(
Arrays.stream(families)
.map(fam -> ColumnFamilyDescriptorBuilder.newBuilder(Bytes.toBytes(fam)).build()) .map(fam -> ColumnFamilyDescriptorBuilder.newBuilder(Bytes.toBytes(fam)).build())
.collect(Collectors.toList())) .collect(Collectors.toList()))
.build(); .build();
@ -346,17 +348,20 @@ public class TestTruncateTableProcedure extends TestTableDDLProcedureBase {
new CreateTableProcedure(procExec.getEnvironment(), htd, regions)); new CreateTableProcedure(procExec.getEnvironment(), htd, regions));
ProcedureTestingUtility.assertProcNotFailed(procExec.getResult(procId)); ProcedureTestingUtility.assertProcNotFailed(procExec.getResult(procId));
splitAndTruncate(tableName, regions, 3); splitAndTruncate(tableName, regions);
} }
private void splitAndTruncate(TableName tableName, RegionInfo[] regions, int regionReplication) private void splitAndTruncate(TableName tableName, RegionInfo[] regions) throws IOException,
throws IOException, InterruptedException { InterruptedException {
// split a region // split a region
UTIL.getAdmin().split(tableName, new byte[] { '0' }); UTIL.getAdmin().split(tableName, new byte[]{'0'});
UTIL.waitUntilAllRegionsAssigned(tableName);
// wait until split really happens // wait until split really happens
UTIL.waitFor(60000, while (UTIL.getAdmin().getRegions(tableName).size() <= regions.length) {
() -> UTIL.getAdmin().getRegions(tableName).size() > regions.length * regionReplication); Thread.sleep(50);
}
// disable the table // disable the table
UTIL.getAdmin().disableTable(tableName); UTIL.getAdmin().disableTable(tableName);
@ -368,8 +373,5 @@ public class TestTruncateTableProcedure extends TestTableDDLProcedureBase {
ProcedureTestingUtility.assertProcNotFailed(procExec, procId); ProcedureTestingUtility.assertProcNotFailed(procExec, procId);
UTIL.waitUntilAllRegionsAssigned(tableName); UTIL.waitUntilAllRegionsAssigned(tableName);
// confirm that we have the correct number of regions
assertEquals((regions.length + 1) * regionReplication,
UTIL.getAdmin().getRegions(tableName).size());
} }
} }