Skip to content

Commit 1fd426b

Browse files
HBASE-30169 add assertion for split validation
1 parent 340e627 commit 1fd426b

1 file changed

Lines changed: 6 additions & 0 deletions

File tree

hbase-server/src/test/java/org/apache/hadoop/hbase/master/assignment/TestRegionSplit.java

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -160,6 +160,8 @@ public void testSplitTableRegion() throws Exception {
160160
public void testRITWithSplitTableRegion() throws Exception {
161161
final TableName tableName = TableName.valueOf(testMethodName);
162162
final ProcedureExecutor<MasterProcedureEnv> procExec = getMasterProcedureExecutor();
163+
// Disable CatalogJanitor to keep the split region in hbase:meta throughout the test
164+
UTIL.getHBaseCluster().getMaster().getCatalogJanitor().setEnabled(false);
163165

164166
RegionInfo[] regions =
165167
MasterProcedureTestingUtility.createTable(procExec, tableName, null, columnFamilyName);
@@ -184,6 +186,8 @@ public void testRITWithSplitTableRegion() throws Exception {
184186
assertEquals(2, UTIL.getHBaseCluster().getRegions(tableName).size(), "not able to split table");
185187
assertFalse(AssignmentTestingUtil.isRegionInTransition(regions[0],
186188
UTIL.getHBaseCluster().getMaster().getAssignmentManager()));
189+
assertTrue(UTIL.getHBaseCluster().getMaster().getAssignmentManager().getRegionStates()
190+
.getOrCreateRegionStateNode(regions[0]).isSplit());
187191
// As there are only 3 RS, start one more RS before expiring one
188192
UTIL.getHBaseCluster().startRegionServer();
189193

@@ -205,6 +209,8 @@ public void testRITWithSplitTableRegion() throws Exception {
205209

206210
assertFalse(AssignmentTestingUtil.isRegionInTransition(regions[0],
207211
UTIL.getHBaseCluster().getMaster().getAssignmentManager()));
212+
assertTrue(UTIL.getHBaseCluster().getMaster().getAssignmentManager().getRegionStates()
213+
.getOrCreateRegionStateNode(regions[0]).isSplit());
208214
}
209215

210216
@Test

0 commit comments

Comments
 (0)