HBASE-15746 Remove extra RegionCoprocessor preClose() in RSRpcServices#closeRegion (Stephen Yuan Jiang)
Amending-Author: Andrew Purtell <apurt...@apache.org> Project: http://git-wip-us.apache.org/repos/asf/hbase/repo Commit: http://git-wip-us.apache.org/repos/asf/hbase/commit/13dee1cf Tree: http://git-wip-us.apache.org/repos/asf/hbase/tree/13dee1cf Diff: http://git-wip-us.apache.org/repos/asf/hbase/diff/13dee1cf Branch: refs/heads/0.98 Commit: 13dee1cf467e66147c24a7cff158eb6dedf9f3a9 Parents: 947e74e Author: Stephen Yuan Jiang <syuanjiang...@gmail.com> Authored: Tue Jun 14 13:30:41 2016 -0700 Committer: Andrew Purtell <apurt...@apache.org> Committed: Fri Jun 24 10:45:01 2016 -0700 ---------------------------------------------------------------------- .../org/apache/hadoop/hbase/regionserver/HRegionServer.java | 6 ------ 1 file changed, 6 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/hbase/blob/13dee1cf/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java ---------------------------------------------------------------------- diff --git a/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java b/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java index 9df2a2c..d238fef 100644 --- a/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java +++ b/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java @@ -4129,12 +4129,6 @@ public class HRegionServer implements ClientProtos.ClientService.BlockingInterfa } final String encodedRegionName = ProtobufUtil.getRegionEncodedName(request.getRegion()); - // Can be null if we're calling close on a region that's not online - final HRegion region = this.getFromOnlineRegions(encodedRegionName); - if ((region != null) && (region .getCoprocessorHost() != null)) { - region.getCoprocessorHost().preClose(false); - } - requestCount.increment(); LOG.info("Close " + encodedRegionName + ", via zk=" + (zk ? "yes" : "no") + ", znode version=" + versionOfClosingNode + ", on " + sn);