HBASE-24821 simplify the logic of getRegionInfo in TestFlushFromClient to reduce redundancy code

bsglz · virajjasani · commit c39cad220d99 · 2020-08-05T19:59:50.000+05:30
Closes #2200 Signed-off-by: Viraj Jasani <vjasani@apache.org>
diff --git a/hbase-server/src/test/java/org/apache/hadoop/hbase/client/TestFlushFromClient.java b/hbase-server/src/test/java/org/apache/hadoop/hbase/client/TestFlushFromClient.java
@@ -194,11 +194,7 @@ public void testAsyncFlushRegionServer() throws Exception {
   }
 
   private List<HRegion> getRegionInfo() {
-    return TEST_UTIL.getHBaseCluster().getLiveRegionServerThreads().stream()
-      .map(JVMClusterUtil.RegionServerThread::getRegionServer)
-      .flatMap(r -> r.getRegions().stream())
-      .filter(r -> r.getTableDescriptor().getTableName().equals(tableName))
-      .collect(Collectors.toList());
+    return TEST_UTIL.getHBaseCluster().getRegions(tableName);
   }
 
   private List<HRegion> getRegionInfo(HRegionServer rs) {

Original file line number	Diff line number	Diff line change
`@@ -194,11 +194,7 @@ public void testAsyncFlushRegionServer() throws Exception {`
`194`	`194`	`}`
`195`	`195`
`196`	`196`	`private List<HRegion> getRegionInfo() {`
`197`		`- return TEST_UTIL.getHBaseCluster().getLiveRegionServerThreads().stream()`
`198`		`- .map(JVMClusterUtil.RegionServerThread::getRegionServer)`
`199`		`- .flatMap(r -> r.getRegions().stream())`
`200`		`- .filter(r -> r.getTableDescriptor().getTableName().equals(tableName))`
`201`		`- .collect(Collectors.toList());`
	`197`	`+ return TEST_UTIL.getHBaseCluster().getRegions(tableName);`
`202`	`198`	`}`
`203`	`199`
`204`	`200`	`private List<HRegion> getRegionInfo(HRegionServer rs) {`