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 2016/10/12 23:08:32 UTC
hbase git commit: HBASE-16746 The log of “region close” should differ from “region move” (ChiaPing Tsai)
Repository: hbase
Updated Branches:
refs/heads/master c6289f197 -> dfb2a800c
HBASE-16746 The log of \u201cregion close\u201d should differ from \u201cregion move\u201d (ChiaPing Tsai)
Project: http://git-wip-us.apache.org/repos/asf/hbase/repo
Commit: http://git-wip-us.apache.org/repos/asf/hbase/commit/dfb2a800
Tree: http://git-wip-us.apache.org/repos/asf/hbase/tree/dfb2a800
Diff: http://git-wip-us.apache.org/repos/asf/hbase/diff/dfb2a800
Branch: refs/heads/master
Commit: dfb2a800c43b30c326dbdcec673387f7033f2092
Parents: c6289f1
Author: Michael Stack <st...@apache.org>
Authored: Wed Oct 12 16:08:25 2016 -0700
Committer: Michael Stack <st...@apache.org>
Committed: Wed Oct 12 16:08:25 2016 -0700
----------------------------------------------------------------------
.../org/apache/hadoop/hbase/regionserver/RSRpcServices.java | 6 +++++-
1 file changed, 5 insertions(+), 1 deletion(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/hbase/blob/dfb2a800/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/RSRpcServices.java
----------------------------------------------------------------------
diff --git a/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/RSRpcServices.java b/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/RSRpcServices.java
index 8f57710..09c7fb9 100644
--- a/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/RSRpcServices.java
+++ b/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/RSRpcServices.java
@@ -1348,7 +1348,11 @@ public class RSRpcServices implements HBaseRPCErrorHandler,
final String encodedRegionName = ProtobufUtil.getRegionEncodedName(request.getRegion());
requestCount.increment();
- LOG.info("Close " + encodedRegionName + ", moving to " + sn);
+ if (sn == null) {
+ LOG.info("Close " + encodedRegionName + " without moving");
+ } else {
+ LOG.info("Close " + encodedRegionName + ", moving to " + sn);
+ }
boolean closed = regionServer.closeRegion(encodedRegionName, false, sn);
CloseRegionResponse.Builder builder = CloseRegionResponse.newBuilder().setClosed(closed);
return builder.build();