Apache9 commented on a change in pull request #1811:
URL: https://github.com/apache/hbase/pull/1811#discussion_r433584900
##########
File path:
hbase-server/src/main/java/org/apache/hadoop/hbase/master/HMaster.java
##########
@@ -1563,7 +1564,7 @@ protected void stopServiceThreads() {
private void createProcedureExecutor() throws IOException {
MasterProcedureEnv procEnv = new MasterProcedureEnv(this);
procedureStore =
- new RegionProcedureStore(this, localStore, new
MasterProcedureEnv.FsUtilsLeaseRecovery(this));
+ new RegionProcedureStore(this, masterRegion, new
MasterProcedureEnv.FsUtilsLeaseRecovery(this));
Review comment:
The idea is to use different families. There is a known risk that, if
someone stores a lot data in one of the families, it will slow down the start
up of the whole HMaster, even if it is not necessary. We should document this
in our ref guide. Can be a follow on issue?
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
