You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by vj...@apache.org on 2020/08/05 14:31:00 UTC

[hbase] branch master updated: HBASE-24821 simplify the logic of getRegionInfo in TestFlushFromClient to reduce redundancy code

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

vjasani 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 c39cad2  HBASE-24821 simplify the logic of getRegionInfo in TestFlushFromClient to reduce redundancy code
c39cad2 is described below

commit c39cad220d998dcfec42f6b8d36d0268c3a92516
Author: Zheng Wang <18...@qq.com>
AuthorDate: Wed Aug 5 19:56:09 2020 +0530

    HBASE-24821 simplify the logic of getRegionInfo in TestFlushFromClient to reduce redundancy code
    
    Closes #2200
    
    Signed-off-by: Viraj Jasani <vj...@apache.org>
---
 .../java/org/apache/hadoop/hbase/client/TestFlushFromClient.java    | 6 +-----
 1 file changed, 1 insertion(+), 5 deletions(-)

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
index b0fb0b7..3085296 100644
--- 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 class TestFlushFromClient {
   }
 
   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) {