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 2019/08/25 03:56:25 UTC
[hbase] branch master updated: HBASE-22904 NPE occurs when RS send
space quota usage report during HMaster init (#529)
This is an automated email from the ASF dual-hosted git repository.
stack 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 00581ac HBASE-22904 NPE occurs when RS send space quota usage report during HMaster init (#529)
00581ac is described below
commit 00581ac911fe012f31e95af03d3943bea9b93205
Author: Pankaj <pa...@huawei.com>
AuthorDate: Sun Aug 25 09:26:20 2019 +0530
HBASE-22904 NPE occurs when RS send space quota usage report during HMaster init (#529)
* NPE occurs when RS send space quota usage report during HMaster init
* Added the debug log
Signed-off-by: Sakthi <sa...@apache.org>
Signed-off-by: stack <st...@apache.org>
---
.../apache/hadoop/hbase/master/MasterRpcServices.java | 16 ++++++++++++----
1 file changed, 12 insertions(+), 4 deletions(-)
diff --git a/hbase-server/src/main/java/org/apache/hadoop/hbase/master/MasterRpcServices.java b/hbase-server/src/main/java/org/apache/hadoop/hbase/master/MasterRpcServices.java
index c8caea7..e1c21c6 100644
--- a/hbase-server/src/main/java/org/apache/hadoop/hbase/master/MasterRpcServices.java
+++ b/hbase-server/src/main/java/org/apache/hadoop/hbase/master/MasterRpcServices.java
@@ -2227,10 +2227,15 @@ public class MasterRpcServices extends RSRpcServices
return RegionSpaceUseReportResponse.newBuilder().build();
}
MasterQuotaManager quotaManager = this.master.getMasterQuotaManager();
- final long now = EnvironmentEdgeManager.currentTime();
- for (RegionSpaceUse report : request.getSpaceUseList()) {
- quotaManager.addRegionSize(ProtobufUtil.toRegionInfo(
- report.getRegionInfo()), report.getRegionSize(), now);
+ if (quotaManager != null) {
+ final long now = EnvironmentEdgeManager.currentTime();
+ for (RegionSpaceUse report : request.getSpaceUseList()) {
+ quotaManager.addRegionSize(ProtobufUtil.toRegionInfo(report.getRegionInfo()),
+ report.getRegionSize(), now);
+ }
+ } else {
+ LOG.debug(
+ "Received region space usage report but HMaster is not ready to process it, skipping");
}
return RegionSpaceUseReportResponse.newBuilder().build();
} catch (Exception e) {
@@ -2265,6 +2270,9 @@ public class MasterRpcServices extends RSRpcServices
.setSize(tableSize.getValue()).build());
}
return builder.build();
+ } else {
+ LOG.debug(
+ "Received space quota region size report but HMaster is not ready to process it, skipping");
}
return builder.build();
} catch (Exception e) {