You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by st...@apache.org on 2020/04/05 20:24:06 UTC

[hbase] branch branch-2.3 updated: HBASE-24105 [Flakey Test] regionserver.TestRegionReplicas (#1425)

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

stack pushed a commit to branch branch-2.3
in repository https://gitbox.apache.org/repos/asf/hbase.git


The following commit(s) were added to refs/heads/branch-2.3 by this push:
     new 5a67fb8  HBASE-24105 [Flakey Test] regionserver.TestRegionReplicas (#1425)
5a67fb8 is described below

commit 5a67fb8c832634353bcb4d5c71a1fbbf63d8e6f5
Author: huaxiangsun <hu...@apache.org>
AuthorDate: Sun Apr 5 13:20:23 2020 -0700

    HBASE-24105 [Flakey Test] regionserver.TestRegionReplicas (#1425)
    
    Co-authored-by: Huaxiang Sun <hu...@apache.com>
    Signed-off-by: stack <st...@apache.org>
---
 .../apache/hadoop/hbase/regionserver/handler/CloseRegionHandler.java  | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/handler/CloseRegionHandler.java b/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/handler/CloseRegionHandler.java
index 6f43042..2b84dfd 100644
--- a/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/handler/CloseRegionHandler.java
+++ b/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/handler/CloseRegionHandler.java
@@ -117,9 +117,7 @@ public class CloseRegionHandler extends EventHandler {
         HConstants.NO_SEQNUM, Procedure.NO_PROC_ID, -1, regionInfo));
 
       // Done!  Region is closed on this RS
-      this.rsServices.getRegionsInTransitionInRS().
-        remove(this.regionInfo.getEncodedNameAsBytes(), Boolean.FALSE);
-      LOG.debug("Closed {}" + region.getRegionInfo().getRegionNameAsString());
+      LOG.debug("Closed {}", region.getRegionInfo().getRegionNameAsString());
     } finally {
       // Clear any reference in getServer().getRegionsInTransitionInRS() on success or failure,
       // since a reference was added before this CRH was invoked. If we don't clear it, it can