You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by gi...@apache.org on 2018/01/30 15:18:41 UTC

[33/44] hbase-site git commit: Published site at .

http://git-wip-us.apache.org/repos/asf/hbase-site/blob/a3fb6d74/devapidocs/src-html/org/apache/hadoop/hbase/master/HMaster.RedirectServlet.html
----------------------------------------------------------------------
diff --git a/devapidocs/src-html/org/apache/hadoop/hbase/master/HMaster.RedirectServlet.html b/devapidocs/src-html/org/apache/hadoop/hbase/master/HMaster.RedirectServlet.html
index ce3e039..4a87b9d 100644
--- a/devapidocs/src-html/org/apache/hadoop/hbase/master/HMaster.RedirectServlet.html
+++ b/devapidocs/src-html/org/apache/hadoop/hbase/master/HMaster.RedirectServlet.html
@@ -6,7 +6,7 @@
 </head>
 <body>
 <div class="sourceContainer">
-<pre><span class="sourceLineNo">001</span>/**<a name="line.1"></a>
+<pre><span class="sourceLineNo">001</span>/*<a name="line.1"></a>
 <span class="sourceLineNo">002</span> * Licensed to the Apache Software Foundation (ASF) under one<a name="line.2"></a>
 <span class="sourceLineNo">003</span> * or more contributor license agreements.  See the NOTICE file<a name="line.3"></a>
 <span class="sourceLineNo">004</span> * distributed with this work for additional information<a name="line.4"></a>
@@ -825,13 +825,13 @@
 <span class="sourceLineNo">817</span><a name="line.817"></a>
 <span class="sourceLineNo">818</span>    this.serverManager = createServerManager(this);<a name="line.818"></a>
 <span class="sourceLineNo">819</span><a name="line.819"></a>
-<span class="sourceLineNo">820</span>    this.tableStateManager = new TableStateManager(this);<a name="line.820"></a>
-<span class="sourceLineNo">821</span><a name="line.821"></a>
-<span class="sourceLineNo">822</span>    status.setStatus("Initializing ZK system trackers");<a name="line.822"></a>
-<span class="sourceLineNo">823</span>    initializeZKBasedSystemTrackers();<a name="line.823"></a>
-<span class="sourceLineNo">824</span><a name="line.824"></a>
-<span class="sourceLineNo">825</span>    // Set Master as active now after we've setup zk with stuff like whether cluster is up or not.<a name="line.825"></a>
-<span class="sourceLineNo">826</span>    // RegionServers won't come up if the cluster status is not up.<a name="line.826"></a>
+<span class="sourceLineNo">820</span>    // This manager is started AFTER hbase:meta is confirmed on line.<a name="line.820"></a>
+<span class="sourceLineNo">821</span>    // See inside metaBootstrap.recoverMeta(); below. Shouldn't be so cryptic!<a name="line.821"></a>
+<span class="sourceLineNo">822</span>    this.tableStateManager = new TableStateManager(this);<a name="line.822"></a>
+<span class="sourceLineNo">823</span><a name="line.823"></a>
+<span class="sourceLineNo">824</span>    status.setStatus("Initializing ZK system trackers");<a name="line.824"></a>
+<span class="sourceLineNo">825</span>    initializeZKBasedSystemTrackers();<a name="line.825"></a>
+<span class="sourceLineNo">826</span>    // Set ourselves as active Master now our claim has succeeded up in zk.<a name="line.826"></a>
 <span class="sourceLineNo">827</span>    this.activeMaster = true;<a name="line.827"></a>
 <span class="sourceLineNo">828</span><a name="line.828"></a>
 <span class="sourceLineNo">829</span>    // This is for backwards compatibility<a name="line.829"></a>
@@ -874,2697 +874,2693 @@
 <span class="sourceLineNo">866</span>    this.balancer.setClusterMetrics(getClusterMetricsWithoutCoprocessor());<a name="line.866"></a>
 <span class="sourceLineNo">867</span>    this.balancer.initialize();<a name="line.867"></a>
 <span class="sourceLineNo">868</span><a name="line.868"></a>
-<span class="sourceLineNo">869</span>    // Check if master is shutting down because of some issue<a name="line.869"></a>
-<span class="sourceLineNo">870</span>    // in initializing the regionserver or the balancer.<a name="line.870"></a>
-<span class="sourceLineNo">871</span>    if (isStopped()) return;<a name="line.871"></a>
-<span class="sourceLineNo">872</span><a name="line.872"></a>
-<span class="sourceLineNo">873</span>    // Make sure meta assigned before proceeding.<a name="line.873"></a>
-<span class="sourceLineNo">874</span>    status.setStatus("Recovering  Meta Region");<a name="line.874"></a>
-<span class="sourceLineNo">875</span><a name="line.875"></a>
-<span class="sourceLineNo">876</span>    // we recover hbase:meta region servers inside master initialization and<a name="line.876"></a>
-<span class="sourceLineNo">877</span>    // handle other failed servers in SSH in order to start up master node ASAP<a name="line.877"></a>
-<span class="sourceLineNo">878</span>    MasterMetaBootstrap metaBootstrap = createMetaBootstrap(this, status);<a name="line.878"></a>
-<span class="sourceLineNo">879</span>    metaBootstrap.recoverMeta();<a name="line.879"></a>
-<span class="sourceLineNo">880</span><a name="line.880"></a>
-<span class="sourceLineNo">881</span>    // check if master is shutting down because above assignMeta could return even hbase:meta isn't<a name="line.881"></a>
-<span class="sourceLineNo">882</span>    // assigned when master is shutting down<a name="line.882"></a>
-<span class="sourceLineNo">883</span>    if (isStopped()) return;<a name="line.883"></a>
-<span class="sourceLineNo">884</span><a name="line.884"></a>
-<span class="sourceLineNo">885</span>    //Initialize after meta as it scans meta<a name="line.885"></a>
-<span class="sourceLineNo">886</span>    if (favoredNodesManager != null) {<a name="line.886"></a>
-<span class="sourceLineNo">887</span>      SnapshotOfRegionAssignmentFromMeta snapshotOfRegionAssignment =<a name="line.887"></a>
-<span class="sourceLineNo">888</span>          new SnapshotOfRegionAssignmentFromMeta(getConnection());<a name="line.888"></a>
-<span class="sourceLineNo">889</span>      snapshotOfRegionAssignment.initialize();<a name="line.889"></a>
-<span class="sourceLineNo">890</span>      favoredNodesManager.initialize(snapshotOfRegionAssignment);<a name="line.890"></a>
-<span class="sourceLineNo">891</span>    }<a name="line.891"></a>
-<span class="sourceLineNo">892</span><a name="line.892"></a>
-<span class="sourceLineNo">893</span>    // migrating existent table state from zk, so splitters<a name="line.893"></a>
-<span class="sourceLineNo">894</span>    // and recovery process treat states properly.<a name="line.894"></a>
-<span class="sourceLineNo">895</span>    for (Map.Entry&lt;TableName, TableState.State&gt; entry : ZKDataMigrator<a name="line.895"></a>
-<span class="sourceLineNo">896</span>        .queryForTableStates(getZooKeeper()).entrySet()) {<a name="line.896"></a>
-<span class="sourceLineNo">897</span>      LOG.info("Converting state from zk to new states:" + entry);<a name="line.897"></a>
-<span class="sourceLineNo">898</span>      tableStateManager.setTableState(entry.getKey(), entry.getValue());<a name="line.898"></a>
-<span class="sourceLineNo">899</span>    }<a name="line.899"></a>
-<span class="sourceLineNo">900</span>    ZKUtil.deleteChildrenRecursively(getZooKeeper(), getZooKeeper().znodePaths.tableZNode);<a name="line.900"></a>
+<span class="sourceLineNo">869</span>    // Make sure meta assigned before proceeding.<a name="line.869"></a>
+<span class="sourceLineNo">870</span>    status.setStatus("Recovering  Meta Region");<a name="line.870"></a>
+<span class="sourceLineNo">871</span><a name="line.871"></a>
+<span class="sourceLineNo">872</span>    // we recover hbase:meta region servers inside master initialization and<a name="line.872"></a>
+<span class="sourceLineNo">873</span>    // handle other failed servers in SSH in order to start up master node ASAP<a name="line.873"></a>
+<span class="sourceLineNo">874</span>    MasterMetaBootstrap metaBootstrap = createMetaBootstrap(this, status);<a name="line.874"></a>
+<span class="sourceLineNo">875</span>    metaBootstrap.recoverMeta();<a name="line.875"></a>
+<span class="sourceLineNo">876</span><a name="line.876"></a>
+<span class="sourceLineNo">877</span><a name="line.877"></a>
+<span class="sourceLineNo">878</span>    //Initialize after meta as it scans meta<a name="line.878"></a>
+<span class="sourceLineNo">879</span>    if (favoredNodesManager != null) {<a name="line.879"></a>
+<span class="sourceLineNo">880</span>      SnapshotOfRegionAssignmentFromMeta snapshotOfRegionAssignment =<a name="line.880"></a>
+<span class="sourceLineNo">881</span>          new SnapshotOfRegionAssignmentFromMeta(getConnection());<a name="line.881"></a>
+<span class="sourceLineNo">882</span>      snapshotOfRegionAssignment.initialize();<a name="line.882"></a>
+<span class="sourceLineNo">883</span>      favoredNodesManager.initialize(snapshotOfRegionAssignment);<a name="line.883"></a>
+<span class="sourceLineNo">884</span>    }<a name="line.884"></a>
+<span class="sourceLineNo">885</span><a name="line.885"></a>
+<span class="sourceLineNo">886</span>    // migrating existent table state from zk, so splitters<a name="line.886"></a>
+<span class="sourceLineNo">887</span>    // and recovery process treat states properly.<a name="line.887"></a>
+<span class="sourceLineNo">888</span>    for (Map.Entry&lt;TableName, TableState.State&gt; entry : ZKDataMigrator<a name="line.888"></a>
+<span class="sourceLineNo">889</span>        .queryForTableStates(getZooKeeper()).entrySet()) {<a name="line.889"></a>
+<span class="sourceLineNo">890</span>      LOG.info("Converting state from zk to new states:" + entry);<a name="line.890"></a>
+<span class="sourceLineNo">891</span>      tableStateManager.setTableState(entry.getKey(), entry.getValue());<a name="line.891"></a>
+<span class="sourceLineNo">892</span>    }<a name="line.892"></a>
+<span class="sourceLineNo">893</span>    ZKUtil.deleteChildrenRecursively(getZooKeeper(), getZooKeeper().znodePaths.tableZNode);<a name="line.893"></a>
+<span class="sourceLineNo">894</span><a name="line.894"></a>
+<span class="sourceLineNo">895</span>    status.setStatus("Submitting log splitting work for previously failed region servers");<a name="line.895"></a>
+<span class="sourceLineNo">896</span>    metaBootstrap.processDeadServers();<a name="line.896"></a>
+<span class="sourceLineNo">897</span><a name="line.897"></a>
+<span class="sourceLineNo">898</span>    // Fix up assignment manager status<a name="line.898"></a>
+<span class="sourceLineNo">899</span>    status.setStatus("Starting assignment manager");<a name="line.899"></a>
+<span class="sourceLineNo">900</span>    this.assignmentManager.joinCluster();<a name="line.900"></a>
 <span class="sourceLineNo">901</span><a name="line.901"></a>
-<span class="sourceLineNo">902</span>    status.setStatus("Submitting log splitting work for previously failed region servers");<a name="line.902"></a>
-<span class="sourceLineNo">903</span>    metaBootstrap.processDeadServers();<a name="line.903"></a>
+<span class="sourceLineNo">902</span>    // set cluster status again after user regions are assigned<a name="line.902"></a>
+<span class="sourceLineNo">903</span>    this.balancer.setClusterMetrics(getClusterMetricsWithoutCoprocessor());<a name="line.903"></a>
 <span class="sourceLineNo">904</span><a name="line.904"></a>
-<span class="sourceLineNo">905</span>    // Fix up assignment manager status<a name="line.905"></a>
-<span class="sourceLineNo">906</span>    status.setStatus("Starting assignment manager");<a name="line.906"></a>
-<span class="sourceLineNo">907</span>    this.assignmentManager.joinCluster();<a name="line.907"></a>
-<span class="sourceLineNo">908</span><a name="line.908"></a>
-<span class="sourceLineNo">909</span>    // set cluster status again after user regions are assigned<a name="line.909"></a>
-<span class="sourceLineNo">910</span>    this.balancer.setClusterMetrics(getClusterMetricsWithoutCoprocessor());<a name="line.910"></a>
-<span class="sourceLineNo">911</span><a name="line.911"></a>
-<span class="sourceLineNo">912</span>    // Start balancer and meta catalog janitor after meta and regions have been assigned.<a name="line.912"></a>
-<span class="sourceLineNo">913</span>    status.setStatus("Starting balancer and catalog janitor");<a name="line.913"></a>
-<span class="sourceLineNo">914</span>    this.clusterStatusChore = new ClusterStatusChore(this, balancer);<a name="line.914"></a>
-<span class="sourceLineNo">915</span>    getChoreService().scheduleChore(clusterStatusChore);<a name="line.915"></a>
-<span class="sourceLineNo">916</span>    this.balancerChore = new BalancerChore(this);<a name="line.916"></a>
-<span class="sourceLineNo">917</span>    getChoreService().scheduleChore(balancerChore);<a name="line.917"></a>
-<span class="sourceLineNo">918</span>    this.normalizerChore = new RegionNormalizerChore(this);<a name="line.918"></a>
-<span class="sourceLineNo">919</span>    getChoreService().scheduleChore(normalizerChore);<a name="line.919"></a>
-<span class="sourceLineNo">920</span>    this.catalogJanitorChore = new CatalogJanitor(this);<a name="line.920"></a>
-<span class="sourceLineNo">921</span>    getChoreService().scheduleChore(catalogJanitorChore);<a name="line.921"></a>
-<span class="sourceLineNo">922</span><a name="line.922"></a>
-<span class="sourceLineNo">923</span>    status.setStatus("Starting cluster schema service");<a name="line.923"></a>
-<span class="sourceLineNo">924</span>    initClusterSchemaService();<a name="line.924"></a>
-<span class="sourceLineNo">925</span><a name="line.925"></a>
-<span class="sourceLineNo">926</span>    if (this.cpHost != null) {<a name="line.926"></a>
-<span class="sourceLineNo">927</span>      try {<a name="line.927"></a>
-<span class="sourceLineNo">928</span>        this.cpHost.preMasterInitialization();<a name="line.928"></a>
-<span class="sourceLineNo">929</span>      } catch (IOException e) {<a name="line.929"></a>
-<span class="sourceLineNo">930</span>        LOG.error("Coprocessor preMasterInitialization() hook failed", e);<a name="line.930"></a>
-<span class="sourceLineNo">931</span>      }<a name="line.931"></a>
-<span class="sourceLineNo">932</span>    }<a name="line.932"></a>
-<span class="sourceLineNo">933</span><a name="line.933"></a>
-<span class="sourceLineNo">934</span>    status.markComplete("Initialization successful");<a name="line.934"></a>
-<span class="sourceLineNo">935</span>    LOG.info(String.format("Master has completed initialization %.3fsec",<a name="line.935"></a>
-<span class="sourceLineNo">936</span>       (System.currentTimeMillis() - masterActiveTime) / 1000.0f));<a name="line.936"></a>
-<span class="sourceLineNo">937</span>    this.masterFinishedInitializationTime = System.currentTimeMillis();<a name="line.937"></a>
-<span class="sourceLineNo">938</span>    configurationManager.registerObserver(this.balancer);<a name="line.938"></a>
-<span class="sourceLineNo">939</span>    configurationManager.registerObserver(this.hfileCleaner);<a name="line.939"></a>
-<span class="sourceLineNo">940</span>    configurationManager.registerObserver(this.logCleaner);<a name="line.940"></a>
-<span class="sourceLineNo">941</span><a name="line.941"></a>
-<span class="sourceLineNo">942</span>    // Set master as 'initialized'.<a name="line.942"></a>
-<span class="sourceLineNo">943</span>    setInitialized(true);<a name="line.943"></a>
-<span class="sourceLineNo">944</span><a name="line.944"></a>
-<span class="sourceLineNo">945</span>    assignmentManager.checkIfShouldMoveSystemRegionAsync();<a name="line.945"></a>
-<span class="sourceLineNo">946</span><a name="line.946"></a>
-<span class="sourceLineNo">947</span>    status.setStatus("Assign meta replicas");<a name="line.947"></a>
-<span class="sourceLineNo">948</span>    metaBootstrap.assignMetaReplicas();<a name="line.948"></a>
-<span class="sourceLineNo">949</span><a name="line.949"></a>
-<span class="sourceLineNo">950</span>    status.setStatus("Starting quota manager");<a name="line.950"></a>
-<span class="sourceLineNo">951</span>    initQuotaManager();<a name="line.951"></a>
-<span class="sourceLineNo">952</span>    if (QuotaUtil.isQuotaEnabled(conf)) {<a name="line.952"></a>
-<span class="sourceLineNo">953</span>      // Create the quota snapshot notifier<a name="line.953"></a>
-<span class="sourceLineNo">954</span>      spaceQuotaSnapshotNotifier = createQuotaSnapshotNotifier();<a name="line.954"></a>
-<span class="sourceLineNo">955</span>      spaceQuotaSnapshotNotifier.initialize(getClusterConnection());<a name="line.955"></a>
-<span class="sourceLineNo">956</span>      this.quotaObserverChore = new QuotaObserverChore(this, getMasterMetrics());<a name="line.956"></a>
-<span class="sourceLineNo">957</span>      // Start the chore to read the region FS space reports and act on them<a name="line.957"></a>
-<span class="sourceLineNo">958</span>      getChoreService().scheduleChore(quotaObserverChore);<a name="line.958"></a>
-<span class="sourceLineNo">959</span><a name="line.959"></a>
-<span class="sourceLineNo">960</span>      this.snapshotQuotaChore = new SnapshotQuotaObserverChore(this, getMasterMetrics());<a name="line.960"></a>
-<span class="sourceLineNo">961</span>      // Start the chore to read snapshots and add their usage to table/NS quotas<a name="line.961"></a>
-<span class="sourceLineNo">962</span>      getChoreService().scheduleChore(snapshotQuotaChore);<a name="line.962"></a>
-<span class="sourceLineNo">963</span>    }<a name="line.963"></a>
-<span class="sourceLineNo">964</span><a name="line.964"></a>
-<span class="sourceLineNo">965</span>    // clear the dead servers with same host name and port of online server because we are not<a name="line.965"></a>
-<span class="sourceLineNo">966</span>    // removing dead server with same hostname and port of rs which is trying to check in before<a name="line.966"></a>
-<span class="sourceLineNo">967</span>    // master initialization. See HBASE-5916.<a name="line.967"></a>
-<span class="sourceLineNo">968</span>    this.serverManager.clearDeadServersWithSameHostNameAndPortOfOnlineServer();<a name="line.968"></a>
-<span class="sourceLineNo">969</span><a name="line.969"></a>
-<span class="sourceLineNo">970</span>    // Check and set the znode ACLs if needed in case we are overtaking a non-secure configuration<a name="line.970"></a>
-<span class="sourceLineNo">971</span>    status.setStatus("Checking ZNode ACLs");<a name="line.971"></a>
-<span class="sourceLineNo">972</span>    zooKeeper.checkAndSetZNodeAcls();<a name="line.972"></a>
-<span class="sourceLineNo">973</span><a name="line.973"></a>
-<span class="sourceLineNo">974</span>    status.setStatus("Initializing MOB Cleaner");<a name="line.974"></a>
-<span class="sourceLineNo">975</span>    initMobCleaner();<a name="line.975"></a>
-<span class="sourceLineNo">976</span><a name="line.976"></a>
-<span class="sourceLineNo">977</span>    status.setStatus("Calling postStartMaster coprocessors");<a name="line.977"></a>
-<span class="sourceLineNo">978</span>    if (this.cpHost != null) {<a name="line.978"></a>
-<span class="sourceLineNo">979</span>      // don't let cp initialization errors kill the master<a name="line.979"></a>
-<span class="sourceLineNo">980</span>      try {<a name="line.980"></a>
-<span class="sourceLineNo">981</span>        this.cpHost.postStartMaster();<a name="line.981"></a>
-<span class="sourceLineNo">982</span>      } catch (IOException ioe) {<a name="line.982"></a>
-<span class="sourceLineNo">983</span>        LOG.error("Coprocessor postStartMaster() hook failed", ioe);<a name="line.983"></a>
-<span class="sourceLineNo">984</span>      }<a name="line.984"></a>
-<span class="sourceLineNo">985</span>    }<a name="line.985"></a>
-<span class="sourceLineNo">986</span><a name="line.986"></a>
-<span class="sourceLineNo">987</span>    zombieDetector.interrupt();<a name="line.987"></a>
-<span class="sourceLineNo">988</span>  }<a name="line.988"></a>
-<span class="sourceLineNo">989</span><a name="line.989"></a>
-<span class="sourceLineNo">990</span>  /**<a name="line.990"></a>
-<span class="sourceLineNo">991</span>   * Adds the {@code MasterSpaceQuotaObserver} to the list of configured Master observers to<a name="line.991"></a>
-<span class="sourceLineNo">992</span>   * automatically remove space quotas for a table when that table is deleted.<a name="line.992"></a>
-<span class="sourceLineNo">993</span>   */<a name="line.993"></a>
-<span class="sourceLineNo">994</span>  @VisibleForTesting<a name="line.994"></a>
-<span class="sourceLineNo">995</span>  public void updateConfigurationForSpaceQuotaObserver(Configuration conf) {<a name="line.995"></a>
-<span class="sourceLineNo">996</span>    // We're configured to not delete quotas on table deletion, so we don't need to add the obs.<a name="line.996"></a>
-<span class="sourceLineNo">997</span>    if (!conf.getBoolean(<a name="line.997"></a>
-<span class="sourceLineNo">998</span>          MasterSpaceQuotaObserver.REMOVE_QUOTA_ON_TABLE_DELETE,<a name="line.998"></a>
-<span class="sourceLineNo">999</span>          MasterSpaceQuotaObserver.REMOVE_QUOTA_ON_TABLE_DELETE_DEFAULT)) {<a name="line.999"></a>
-<span class="sourceLineNo">1000</span>      return;<a name="line.1000"></a>
-<span class="sourceLineNo">1001</span>    }<a name="line.1001"></a>
-<span class="sourceLineNo">1002</span>    String[] masterCoprocs = conf.getStrings(CoprocessorHost.MASTER_COPROCESSOR_CONF_KEY);<a name="line.1002"></a>
-<span class="sourceLineNo">1003</span>    final int length = null == masterCoprocs ? 0 : masterCoprocs.length;<a name="line.1003"></a>
-<span class="sourceLineNo">1004</span>    String[] updatedCoprocs = new String[length + 1];<a name="line.1004"></a>
-<span class="sourceLineNo">1005</span>    if (length &gt; 0) {<a name="line.1005"></a>
-<span class="sourceLineNo">1006</span>      System.arraycopy(masterCoprocs, 0, updatedCoprocs, 0, masterCoprocs.length);<a name="line.1006"></a>
-<span class="sourceLineNo">1007</span>    }<a name="line.1007"></a>
-<span class="sourceLineNo">1008</span>    updatedCoprocs[length] = MasterSpaceQuotaObserver.class.getName();<a name="line.1008"></a>
-<span class="sourceLineNo">1009</span>    conf.setStrings(CoprocessorHost.MASTER_COPROCESSOR_CONF_KEY, updatedCoprocs);<a name="line.1009"></a>
-<span class="sourceLineNo">1010</span>  }<a name="line.1010"></a>
-<span class="sourceLineNo">1011</span><a name="line.1011"></a>
-<span class="sourceLineNo">1012</span>  private void initMobCleaner() {<a name="line.1012"></a>
-<span class="sourceLineNo">1013</span>    this.expiredMobFileCleanerChore = new ExpiredMobFileCleanerChore(this);<a name="line.1013"></a>
-<span class="sourceLineNo">1014</span>    getChoreService().scheduleChore(expiredMobFileCleanerChore);<a name="line.1014"></a>
-<span class="sourceLineNo">1015</span><a name="line.1015"></a>
-<span class="sourceLineNo">1016</span>    int mobCompactionPeriod = conf.getInt(MobConstants.MOB_COMPACTION_CHORE_PERIOD,<a name="line.1016"></a>
-<span class="sourceLineNo">1017</span>        MobConstants.DEFAULT_MOB_COMPACTION_CHORE_PERIOD);<a name="line.1017"></a>
-<span class="sourceLineNo">1018</span>    if (mobCompactionPeriod &gt; 0) {<a name="line.1018"></a>
-<span class="sourceLineNo">1019</span>      this.mobCompactChore = new MobCompactionChore(this, mobCompactionPeriod);<a name="line.1019"></a>
-<span class="sourceLineNo">1020</span>      getChoreService().scheduleChore(mobCompactChore);<a name="line.1020"></a>
-<span class="sourceLineNo">1021</span>    } else {<a name="line.1021"></a>
-<span class="sourceLineNo">1022</span>      LOG<a name="line.1022"></a>
-<span class="sourceLineNo">1023</span>        .info("The period is " + mobCompactionPeriod + " seconds, MobCompactionChore is disabled");<a name="line.1023"></a>
-<span class="sourceLineNo">1024</span>    }<a name="line.1024"></a>
-<span class="sourceLineNo">1025</span>    this.mobCompactThread = new MasterMobCompactionThread(this);<a name="line.1025"></a>
-<span class="sourceLineNo">1026</span>  }<a name="line.1026"></a>
-<span class="sourceLineNo">1027</span><a name="line.1027"></a>
-<span class="sourceLineNo">1028</span>  /**<a name="line.1028"></a>
-<span class="sourceLineNo">1029</span>   * Create a {@link MasterMetaBootstrap} instance.<a name="line.1029"></a>
-<span class="sourceLineNo">1030</span>   */<a name="line.1030"></a>
-<span class="sourceLineNo">1031</span>  MasterMetaBootstrap createMetaBootstrap(final HMaster master, final MonitoredTask status) {<a name="line.1031"></a>
-<span class="sourceLineNo">1032</span>    // We put this out here in a method so can do a Mockito.spy and stub it out<a name="line.1032"></a>
-<span class="sourceLineNo">1033</span>    // w/ a mocked up MasterMetaBootstrap.<a name="line.1033"></a>
-<span class="sourceLineNo">1034</span>    return new MasterMetaBootstrap(master, status);<a name="line.1034"></a>
-<span class="sourceLineNo">1035</span>  }<a name="line.1035"></a>
-<span class="sourceLineNo">1036</span><a name="line.1036"></a>
-<span class="sourceLineNo">1037</span>  /**<a name="line.1037"></a>
-<span class="sourceLineNo">1038</span>   * Create a {@link ServerManager} instance.<a name="line.1038"></a>
-<span class="sourceLineNo">1039</span>   */<a name="line.1039"></a>
-<span class="sourceLineNo">1040</span>  ServerManager createServerManager(final MasterServices master) throws IOException {<a name="line.1040"></a>
-<span class="sourceLineNo">1041</span>    // We put this out here in a method so can do a Mockito.spy and stub it out<a name="line.1041"></a>
-<span class="sourceLineNo">1042</span>    // w/ a mocked up ServerManager.<a name="line.1042"></a>
-<span class="sourceLineNo">1043</span>    setupClusterConnection();<a name="line.1043"></a>
-<span class="sourceLineNo">1044</span>    return new ServerManager(master);<a name="line.1044"></a>
-<span class="sourceLineNo">1045</span>  }<a name="line.1045"></a>
-<span class="sourceLineNo">1046</span><a name="line.1046"></a>
-<span class="sourceLineNo">1047</span>  private void waitForRegionServers(final MonitoredTask status)<a name="line.1047"></a>
-<span class="sourceLineNo">1048</span>      throws IOException, InterruptedException {<a name="line.1048"></a>
-<span class="sourceLineNo">1049</span>    this.serverManager.waitForRegionServers(status);<a name="line.1049"></a>
-<span class="sourceLineNo">1050</span>    // Check zk for region servers that are up but didn't register<a name="line.1050"></a>
-<span class="sourceLineNo">1051</span>    for (ServerName sn: this.regionServerTracker.getOnlineServers()) {<a name="line.1051"></a>
-<span class="sourceLineNo">1052</span>      // The isServerOnline check is opportunistic, correctness is handled inside<a name="line.1052"></a>
-<span class="sourceLineNo">1053</span>      if (!this.serverManager.isServerOnline(sn) &amp;&amp;<a name="line.1053"></a>
-<span class="sourceLineNo">1054</span>          serverManager.checkAndRecordNewServer(sn, new ServerLoad(ServerMetricsBuilder.of(sn)))) {<a name="line.1054"></a>
-<span class="sourceLineNo">1055</span>        LOG.info("Registered server found up in zk but who has not yet reported in: " + sn);<a name="line.1055"></a>
-<span class="sourceLineNo">1056</span>      }<a name="line.1056"></a>
-<span class="sourceLineNo">1057</span>    }<a name="line.1057"></a>
-<span class="sourceLineNo">1058</span>  }<a name="line.1058"></a>
-<span class="sourceLineNo">1059</span><a name="line.1059"></a>
-<span class="sourceLineNo">1060</span>  void initClusterSchemaService() throws IOException, InterruptedException {<a name="line.1060"></a>
-<span class="sourceLineNo">1061</span>    this.clusterSchemaService = new ClusterSchemaServiceImpl(this);<a name="line.1061"></a>
-<span class="sourceLineNo">1062</span>    this.clusterSchemaService.startAsync();<a name="line.1062"></a>
-<span class="sourceLineNo">1063</span>    try {<a name="line.1063"></a>
-<span class="sourceLineNo">1064</span>      this.clusterSchemaService.awaitRunning(getConfiguration().getInt(<a name="line.1064"></a>
-<span class="sourceLineNo">1065</span>        HBASE_MASTER_WAIT_ON_SERVICE_IN_SECONDS,<a name="line.1065"></a>
-<span class="sourceLineNo">1066</span>        DEFAULT_HBASE_MASTER_WAIT_ON_SERVICE_IN_SECONDS), TimeUnit.SECONDS);<a name="line.1066"></a>
-<span class="sourceLineNo">1067</span>    } catch (TimeoutException toe) {<a name="line.1067"></a>
-<span class="sourceLineNo">1068</span>      throw new IOException("Timedout starting ClusterSchemaService", toe);<a name="line.1068"></a>
-<span class="sourceLineNo">1069</span>    }<a name="line.1069"></a>
-<span class="sourceLineNo">1070</span>  }<a name="line.1070"></a>
-<span class="sourceLineNo">1071</span><a name="line.1071"></a>
-<span class="sourceLineNo">1072</span>  void initQuotaManager() throws IOException {<a name="line.1072"></a>
-<span class="sourceLineNo">1073</span>    MasterQuotaManager quotaManager = new MasterQuotaManager(this);<a name="line.1073"></a>
-<span class="sourceLineNo">1074</span>    this.assignmentManager.setRegionStateListener(quotaManager);<a name="line.1074"></a>
-<span class="sourceLineNo">1075</span>    quotaManager.start();<a name="line.1075"></a>
-<span class="sourceLineNo">1076</span>    this.quotaManager = quotaManager;<a name="line.1076"></a>
+<span class="sourceLineNo">905</span>    // Start balancer and meta catalog janitor after meta and regions have been assigned.<a name="line.905"></a>
+<span class="sourceLineNo">906</span>    status.setStatus("Starting balancer and catalog janitor");<a name="line.906"></a>
+<span class="sourceLineNo">907</span>    this.clusterStatusChore = new ClusterStatusChore(this, balancer);<a name="line.907"></a>
+<span class="sourceLineNo">908</span>    getChoreService().scheduleChore(clusterStatusChore);<a name="line.908"></a>
+<span class="sourceLineNo">909</span>    this.balancerChore = new BalancerChore(this);<a name="line.909"></a>
+<span class="sourceLineNo">910</span>    getChoreService().scheduleChore(balancerChore);<a name="line.910"></a>
+<span class="sourceLineNo">911</span>    this.normalizerChore = new RegionNormalizerChore(this);<a name="line.911"></a>
+<span class="sourceLineNo">912</span>    getChoreService().scheduleChore(normalizerChore);<a name="line.912"></a>
+<span class="sourceLineNo">913</span>    this.catalogJanitorChore = new CatalogJanitor(this);<a name="line.913"></a>
+<span class="sourceLineNo">914</span>    getChoreService().scheduleChore(catalogJanitorChore);<a name="line.914"></a>
+<span class="sourceLineNo">915</span><a name="line.915"></a>
+<span class="sourceLineNo">916</span>    status.setStatus("Starting cluster schema service");<a name="line.916"></a>
+<span class="sourceLineNo">917</span>    initClusterSchemaService();<a name="line.917"></a>
+<span class="sourceLineNo">918</span><a name="line.918"></a>
+<span class="sourceLineNo">919</span>    if (this.cpHost != null) {<a name="line.919"></a>
+<span class="sourceLineNo">920</span>      try {<a name="line.920"></a>
+<span class="sourceLineNo">921</span>        this.cpHost.preMasterInitialization();<a name="line.921"></a>
+<span class="sourceLineNo">922</span>      } catch (IOException e) {<a name="line.922"></a>
+<span class="sourceLineNo">923</span>        LOG.error("Coprocessor preMasterInitialization() hook failed", e);<a name="line.923"></a>
+<span class="sourceLineNo">924</span>      }<a name="line.924"></a>
+<span class="sourceLineNo">925</span>    }<a name="line.925"></a>
+<span class="sourceLineNo">926</span><a name="line.926"></a>
+<span class="sourceLineNo">927</span>    status.markComplete("Initialization successful");<a name="line.927"></a>
+<span class="sourceLineNo">928</span>    LOG.info(String.format("Master has completed initialization %.3fsec",<a name="line.928"></a>
+<span class="sourceLineNo">929</span>       (System.currentTimeMillis() - masterActiveTime) / 1000.0f));<a name="line.929"></a>
+<span class="sourceLineNo">930</span>    this.masterFinishedInitializationTime = System.currentTimeMillis();<a name="line.930"></a>
+<span class="sourceLineNo">931</span>    configurationManager.registerObserver(this.balancer);<a name="line.931"></a>
+<span class="sourceLineNo">932</span>    configurationManager.registerObserver(this.hfileCleaner);<a name="line.932"></a>
+<span class="sourceLineNo">933</span>    configurationManager.registerObserver(this.logCleaner);<a name="line.933"></a>
+<span class="sourceLineNo">934</span>    // Set master as 'initialized'.<a name="line.934"></a>
+<span class="sourceLineNo">935</span>    setInitialized(true);<a name="line.935"></a>
+<span class="sourceLineNo">936</span>    assignmentManager.checkIfShouldMoveSystemRegionAsync();<a name="line.936"></a>
+<span class="sourceLineNo">937</span>    status.setStatus("Assign meta replicas");<a name="line.937"></a>
+<span class="sourceLineNo">938</span>    metaBootstrap.assignMetaReplicas();<a name="line.938"></a>
+<span class="sourceLineNo">939</span>    status.setStatus("Starting quota manager");<a name="line.939"></a>
+<span class="sourceLineNo">940</span>    initQuotaManager();<a name="line.940"></a>
+<span class="sourceLineNo">941</span>    if (QuotaUtil.isQuotaEnabled(conf)) {<a name="line.941"></a>
+<span class="sourceLineNo">942</span>      // Create the quota snapshot notifier<a name="line.942"></a>
+<span class="sourceLineNo">943</span>      spaceQuotaSnapshotNotifier = createQuotaSnapshotNotifier();<a name="line.943"></a>
+<span class="sourceLineNo">944</span>      spaceQuotaSnapshotNotifier.initialize(getClusterConnection());<a name="line.944"></a>
+<span class="sourceLineNo">945</span>      this.quotaObserverChore = new QuotaObserverChore(this, getMasterMetrics());<a name="line.945"></a>
+<span class="sourceLineNo">946</span>      // Start the chore to read the region FS space reports and act on them<a name="line.946"></a>
+<span class="sourceLineNo">947</span>      getChoreService().scheduleChore(quotaObserverChore);<a name="line.947"></a>
+<span class="sourceLineNo">948</span><a name="line.948"></a>
+<span class="sourceLineNo">949</span>      this.snapshotQuotaChore = new SnapshotQuotaObserverChore(this, getMasterMetrics());<a name="line.949"></a>
+<span class="sourceLineNo">950</span>      // Start the chore to read snapshots and add their usage to table/NS quotas<a name="line.950"></a>
+<span class="sourceLineNo">951</span>      getChoreService().scheduleChore(snapshotQuotaChore);<a name="line.951"></a>
+<span class="sourceLineNo">952</span>    }<a name="line.952"></a>
+<span class="sourceLineNo">953</span><a name="line.953"></a>
+<span class="sourceLineNo">954</span>    // clear the dead servers with same host name and port of online server because we are not<a name="line.954"></a>
+<span class="sourceLineNo">955</span>    // removing dead server with same hostname and port of rs which is trying to check in before<a name="line.955"></a>
+<span class="sourceLineNo">956</span>    // master initialization. See HBASE-5916.<a name="line.956"></a>
+<span class="sourceLineNo">957</span>    this.serverManager.clearDeadServersWithSameHostNameAndPortOfOnlineServer();<a name="line.957"></a>
+<span class="sourceLineNo">958</span><a name="line.958"></a>
+<span class="sourceLineNo">959</span>    // Check and set the znode ACLs if needed in case we are overtaking a non-secure configuration<a name="line.959"></a>
+<span class="sourceLineNo">960</span>    status.setStatus("Checking ZNode ACLs");<a name="line.960"></a>
+<span class="sourceLineNo">961</span>    zooKeeper.checkAndSetZNodeAcls();<a name="line.961"></a>
+<span class="sourceLineNo">962</span><a name="line.962"></a>
+<span class="sourceLineNo">963</span>    status.setStatus("Initializing MOB Cleaner");<a name="line.963"></a>
+<span class="sourceLineNo">964</span>    initMobCleaner();<a name="line.964"></a>
+<span class="sourceLineNo">965</span><a name="line.965"></a>
+<span class="sourceLineNo">966</span>    status.setStatus("Calling postStartMaster coprocessors");<a name="line.966"></a>
+<span class="sourceLineNo">967</span>    if (this.cpHost != null) {<a name="line.967"></a>
+<span class="sourceLineNo">968</span>      // don't let cp initialization errors kill the master<a name="line.968"></a>
+<span class="sourceLineNo">969</span>      try {<a name="line.969"></a>
+<span class="sourceLineNo">970</span>        this.cpHost.postStartMaster();<a name="line.970"></a>
+<span class="sourceLineNo">971</span>      } catch (IOException ioe) {<a name="line.971"></a>
+<span class="sourceLineNo">972</span>        LOG.error("Coprocessor postStartMaster() hook failed", ioe);<a name="line.972"></a>
+<span class="sourceLineNo">973</span>      }<a name="line.973"></a>
+<span class="sourceLineNo">974</span>    }<a name="line.974"></a>
+<span class="sourceLineNo">975</span><a name="line.975"></a>
+<span class="sourceLineNo">976</span>    zombieDetector.interrupt();<a name="line.976"></a>
+<span class="sourceLineNo">977</span>  }<a name="line.977"></a>
+<span class="sourceLineNo">978</span><a name="line.978"></a>
+<span class="sourceLineNo">979</span>  /**<a name="line.979"></a>
+<span class="sourceLineNo">980</span>   * Adds the {@code MasterSpaceQuotaObserver} to the list of configured Master observers to<a name="line.980"></a>
+<span class="sourceLineNo">981</span>   * automatically remove space quotas for a table when that table is deleted.<a name="line.981"></a>
+<span class="sourceLineNo">982</span>   */<a name="line.982"></a>
+<span class="sourceLineNo">983</span>  @VisibleForTesting<a name="line.983"></a>
+<span class="sourceLineNo">984</span>  public void updateConfigurationForSpaceQuotaObserver(Configuration conf) {<a name="line.984"></a>
+<span class="sourceLineNo">985</span>    // We're configured to not delete quotas on table deletion, so we don't need to add the obs.<a name="line.985"></a>
+<span class="sourceLineNo">986</span>    if (!conf.getBoolean(<a name="line.986"></a>
+<span class="sourceLineNo">987</span>          MasterSpaceQuotaObserver.REMOVE_QUOTA_ON_TABLE_DELETE,<a name="line.987"></a>
+<span class="sourceLineNo">988</span>          MasterSpaceQuotaObserver.REMOVE_QUOTA_ON_TABLE_DELETE_DEFAULT)) {<a name="line.988"></a>
+<span class="sourceLineNo">989</span>      return;<a name="line.989"></a>
+<span class="sourceLineNo">990</span>    }<a name="line.990"></a>
+<span class="sourceLineNo">991</span>    String[] masterCoprocs = conf.getStrings(CoprocessorHost.MASTER_COPROCESSOR_CONF_KEY);<a name="line.991"></a>
+<span class="sourceLineNo">992</span>    final int length = null == masterCoprocs ? 0 : masterCoprocs.length;<a name="line.992"></a>
+<span class="sourceLineNo">993</span>    String[] updatedCoprocs = new String[length + 1];<a name="line.993"></a>
+<span class="sourceLineNo">994</span>    if (length &gt; 0) {<a name="line.994"></a>
+<span class="sourceLineNo">995</span>      System.arraycopy(masterCoprocs, 0, updatedCoprocs, 0, masterCoprocs.length);<a name="line.995"></a>
+<span class="sourceLineNo">996</span>    }<a name="line.996"></a>
+<span class="sourceLineNo">997</span>    updatedCoprocs[length] = MasterSpaceQuotaObserver.class.getName();<a name="line.997"></a>
+<span class="sourceLineNo">998</span>    conf.setStrings(CoprocessorHost.MASTER_COPROCESSOR_CONF_KEY, updatedCoprocs);<a name="line.998"></a>
+<span class="sourceLineNo">999</span>  }<a name="line.999"></a>
+<span class="sourceLineNo">1000</span><a name="line.1000"></a>
+<span class="sourceLineNo">1001</span>  private void initMobCleaner() {<a name="line.1001"></a>
+<span class="sourceLineNo">1002</span>    this.expiredMobFileCleanerChore = new ExpiredMobFileCleanerChore(this);<a name="line.1002"></a>
+<span class="sourceLineNo">1003</span>    getChoreService().scheduleChore(expiredMobFileCleanerChore);<a name="line.1003"></a>
+<span class="sourceLineNo">1004</span><a name="line.1004"></a>
+<span class="sourceLineNo">1005</span>    int mobCompactionPeriod = conf.getInt(MobConstants.MOB_COMPACTION_CHORE_PERIOD,<a name="line.1005"></a>
+<span class="sourceLineNo">1006</span>        MobConstants.DEFAULT_MOB_COMPACTION_CHORE_PERIOD);<a name="line.1006"></a>
+<span class="sourceLineNo">1007</span>    if (mobCompactionPeriod &gt; 0) {<a name="line.1007"></a>
+<span class="sourceLineNo">1008</span>      this.mobCompactChore = new MobCompactionChore(this, mobCompactionPeriod);<a name="line.1008"></a>
+<span class="sourceLineNo">1009</span>      getChoreService().scheduleChore(mobCompactChore);<a name="line.1009"></a>
+<span class="sourceLineNo">1010</span>    } else {<a name="line.1010"></a>
+<span class="sourceLineNo">1011</span>      LOG<a name="line.1011"></a>
+<span class="sourceLineNo">1012</span>        .info("The period is " + mobCompactionPeriod + " seconds, MobCompactionChore is disabled");<a name="line.1012"></a>
+<span class="sourceLineNo">1013</span>    }<a name="line.1013"></a>
+<span class="sourceLineNo">1014</span>    this.mobCompactThread = new MasterMobCompactionThread(this);<a name="line.1014"></a>
+<span class="sourceLineNo">1015</span>  }<a name="line.1015"></a>
+<span class="sourceLineNo">1016</span><a name="line.1016"></a>
+<span class="sourceLineNo">1017</span>  /**<a name="line.1017"></a>
+<span class="sourceLineNo">1018</span>   * Create a {@link MasterMetaBootstrap} instance.<a name="line.1018"></a>
+<span class="sourceLineNo">1019</span>   */<a name="line.1019"></a>
+<span class="sourceLineNo">1020</span>  MasterMetaBootstrap createMetaBootstrap(final HMaster master, final MonitoredTask status) {<a name="line.1020"></a>
+<span class="sourceLineNo">1021</span>    // We put this out here in a method so can do a Mockito.spy and stub it out<a name="line.1021"></a>
+<span class="sourceLineNo">1022</span>    // w/ a mocked up MasterMetaBootstrap.<a name="line.1022"></a>
+<span class="sourceLineNo">1023</span>    return new MasterMetaBootstrap(master, status);<a name="line.1023"></a>
+<span class="sourceLineNo">1024</span>  }<a name="line.1024"></a>
+<span class="sourceLineNo">1025</span><a name="line.1025"></a>
+<span class="sourceLineNo">1026</span>  /**<a name="line.1026"></a>
+<span class="sourceLineNo">1027</span>   * Create a {@link ServerManager} instance.<a name="line.1027"></a>
+<span class="sourceLineNo">1028</span>   */<a name="line.1028"></a>
+<span class="sourceLineNo">1029</span>  ServerManager createServerManager(final MasterServices master) throws IOException {<a name="line.1029"></a>
+<span class="sourceLineNo">1030</span>    // We put this out here in a method so can do a Mockito.spy and stub it out<a name="line.1030"></a>
+<span class="sourceLineNo">1031</span>    // w/ a mocked up ServerManager.<a name="line.1031"></a>
+<span class="sourceLineNo">1032</span>    setupClusterConnection();<a name="line.1032"></a>
+<span class="sourceLineNo">1033</span>    return new ServerManager(master);<a name="line.1033"></a>
+<span class="sourceLineNo">1034</span>  }<a name="line.1034"></a>
+<span class="sourceLineNo">1035</span><a name="line.1035"></a>
+<span class="sourceLineNo">1036</span>  private void waitForRegionServers(final MonitoredTask status)<a name="line.1036"></a>
+<span class="sourceLineNo">1037</span>      throws IOException, InterruptedException {<a name="line.1037"></a>
+<span class="sourceLineNo">1038</span>    this.serverManager.waitForRegionServers(status);<a name="line.1038"></a>
+<span class="sourceLineNo">1039</span>    // Check zk for region servers that are up but didn't register<a name="line.1039"></a>
+<span class="sourceLineNo">1040</span>    for (ServerName sn: this.regionServerTracker.getOnlineServers()) {<a name="line.1040"></a>
+<span class="sourceLineNo">1041</span>      // The isServerOnline check is opportunistic, correctness is handled inside<a name="line.1041"></a>
+<span class="sourceLineNo">1042</span>      if (!this.serverManager.isServerOnline(sn) &amp;&amp;<a name="line.1042"></a>
+<span class="sourceLineNo">1043</span>          serverManager.checkAndRecordNewServer(sn, new ServerLoad(ServerMetricsBuilder.of(sn)))) {<a name="line.1043"></a>
+<span class="sourceLineNo">1044</span>        LOG.info("Registered server found up in zk but who has not yet reported in: " + sn);<a name="line.1044"></a>
+<span class="sourceLineNo">1045</span>      }<a name="line.1045"></a>
+<span class="sourceLineNo">1046</span>    }<a name="line.1046"></a>
+<span class="sourceLineNo">1047</span>  }<a name="line.1047"></a>
+<span class="sourceLineNo">1048</span><a name="line.1048"></a>
+<span class="sourceLineNo">1049</span>  void initClusterSchemaService() throws IOException, InterruptedException {<a name="line.1049"></a>
+<span class="sourceLineNo">1050</span>    this.clusterSchemaService = new ClusterSchemaServiceImpl(this);<a name="line.1050"></a>
+<span class="sourceLineNo">1051</span>    this.clusterSchemaService.startAsync();<a name="line.1051"></a>
+<span class="sourceLineNo">1052</span>    try {<a name="line.1052"></a>
+<span class="sourceLineNo">1053</span>      this.clusterSchemaService.awaitRunning(getConfiguration().getInt(<a name="line.1053"></a>
+<span class="sourceLineNo">1054</span>        HBASE_MASTER_WAIT_ON_SERVICE_IN_SECONDS,<a name="line.1054"></a>
+<span class="sourceLineNo">1055</span>        DEFAULT_HBASE_MASTER_WAIT_ON_SERVICE_IN_SECONDS), TimeUnit.SECONDS);<a name="line.1055"></a>
+<span class="sourceLineNo">1056</span>    } catch (TimeoutException toe) {<a name="line.1056"></a>
+<span class="sourceLineNo">1057</span>      throw new IOException("Timedout starting ClusterSchemaService", toe);<a name="line.1057"></a>
+<span class="sourceLineNo">1058</span>    }<a name="line.1058"></a>
+<span class="sourceLineNo">1059</span>  }<a name="line.1059"></a>
+<span class="sourceLineNo">1060</span><a name="line.1060"></a>
+<span class="sourceLineNo">1061</span>  void initQuotaManager() throws IOException {<a name="line.1061"></a>
+<span class="sourceLineNo">1062</span>    MasterQuotaManager quotaManager = new MasterQuotaManager(this);<a name="line.1062"></a>
+<span class="sourceLineNo">1063</span>    this.assignmentManager.setRegionStateListener(quotaManager);<a name="line.1063"></a>
+<span class="sourceLineNo">1064</span>    quotaManager.start();<a name="line.1064"></a>
+<span class="sourceLineNo">1065</span>    this.quotaManager = quotaManager;<a name="line.1065"></a>
+<span class="sourceLineNo">1066</span>  }<a name="line.1066"></a>
+<span class="sourceLineNo">1067</span><a name="line.1067"></a>
+<span class="sourceLineNo">1068</span>  SpaceQuotaSnapshotNotifier createQuotaSnapshotNotifier() {<a name="line.1068"></a>
+<span class="sourceLineNo">1069</span>    SpaceQuotaSnapshotNotifier notifier =<a name="line.1069"></a>
+<span class="sourceLineNo">1070</span>        SpaceQuotaSnapshotNotifierFactory.getInstance().create(getConfiguration());<a name="line.1070"></a>
+<span class="sourceLineNo">1071</span>    return notifier;<a name="line.1071"></a>
+<span class="sourceLineNo">1072</span>  }<a name="line.1072"></a>
+<span class="sourceLineNo">1073</span><a name="line.1073"></a>
+<span class="sourceLineNo">1074</span>  boolean isCatalogJanitorEnabled() {<a name="line.1074"></a>
+<span class="sourceLineNo">1075</span>    return catalogJanitorChore != null ?<a name="line.1075"></a>
+<span class="sourceLineNo">1076</span>      catalogJanitorChore.getEnabled() : false;<a name="line.1076"></a>
 <span class="sourceLineNo">1077</span>  }<a name="line.1077"></a>
 <span class="sourceLineNo">1078</span><a name="line.1078"></a>
-<span class="sourceLineNo">1079</span>  SpaceQuotaSnapshotNotifier createQuotaSnapshotNotifier() {<a name="line.1079"></a>
-<span class="sourceLineNo">1080</span>    SpaceQuotaSnapshotNotifier notifier =<a name="line.1080"></a>
-<span class="sourceLineNo">1081</span>        SpaceQuotaSnapshotNotifierFactory.getInstance().create(getConfiguration());<a name="line.1081"></a>
-<span class="sourceLineNo">1082</span>    return notifier;<a name="line.1082"></a>
-<span class="sourceLineNo">1083</span>  }<a name="line.1083"></a>
-<span class="sourceLineNo">1084</span><a name="line.1084"></a>
-<span class="sourceLineNo">1085</span>  boolean isCatalogJanitorEnabled() {<a name="line.1085"></a>
-<span class="sourceLineNo">1086</span>    return catalogJanitorChore != null ?<a name="line.1086"></a>
-<span class="sourceLineNo">1087</span>      catalogJanitorChore.getEnabled() : false;<a name="line.1087"></a>
-<span class="sourceLineNo">1088</span>  }<a name="line.1088"></a>
+<span class="sourceLineNo">1079</span>  boolean isCleanerChoreEnabled() {<a name="line.1079"></a>
+<span class="sourceLineNo">1080</span>    boolean hfileCleanerFlag = true, logCleanerFlag = true;<a name="line.1080"></a>
+<span class="sourceLineNo">1081</span><a name="line.1081"></a>
+<span class="sourceLineNo">1082</span>    if (hfileCleaner != null) {<a name="line.1082"></a>
+<span class="sourceLineNo">1083</span>      hfileCleanerFlag = hfileCleaner.getEnabled();<a name="line.1083"></a>
+<span class="sourceLineNo">1084</span>    }<a name="line.1084"></a>
+<span class="sourceLineNo">1085</span><a name="line.1085"></a>
+<span class="sourceLineNo">1086</span>    if (logCleaner != null) {<a name="line.1086"></a>
+<span class="sourceLineNo">1087</span>      logCleanerFlag = logCleaner.getEnabled();<a name="line.1087"></a>
+<span class="sourceLineNo">1088</span>    }<a name="line.1088"></a>
 <span class="sourceLineNo">1089</span><a name="line.1089"></a>
-<span class="sourceLineNo">1090</span>  boolean isCleanerChoreEnabled() {<a name="line.1090"></a>
-<span class="sourceLineNo">1091</span>    boolean hfileCleanerFlag = true, logCleanerFlag = true;<a name="line.1091"></a>
+<span class="sourceLineNo">1090</span>    return (hfileCleanerFlag &amp;&amp; logCleanerFlag);<a name="line.1090"></a>
+<span class="sourceLineNo">1091</span>  }<a name="line.1091"></a>
 <span class="sourceLineNo">1092</span><a name="line.1092"></a>
-<span class="sourceLineNo">1093</span>    if (hfileCleaner != null) {<a name="line.1093"></a>
-<span class="sourceLineNo">1094</span>      hfileCleanerFlag = hfileCleaner.getEnabled();<a name="line.1094"></a>
-<span class="sourceLineNo">1095</span>    }<a name="line.1095"></a>
-<span class="sourceLineNo">1096</span><a name="line.1096"></a>
-<span class="sourceLineNo">1097</span>    if (logCleaner != null) {<a name="line.1097"></a>
-<span class="sourceLineNo">1098</span>      logCleanerFlag = logCleaner.getEnabled();<a name="line.1098"></a>
-<span class="sourceLineNo">1099</span>    }<a name="line.1099"></a>
-<span class="sourceLineNo">1100</span><a name="line.1100"></a>
-<span class="sourceLineNo">1101</span>    return (hfileCleanerFlag &amp;&amp; logCleanerFlag);<a name="line.1101"></a>
-<span class="sourceLineNo">1102</span>  }<a name="line.1102"></a>
-<span class="sourceLineNo">1103</span><a name="line.1103"></a>
-<span class="sourceLineNo">1104</span>  @Override<a name="line.1104"></a>
-<span class="sourceLineNo">1105</span>  public TableDescriptors getTableDescriptors() {<a name="line.1105"></a>
-<span class="sourceLineNo">1106</span>    return this.tableDescriptors;<a name="line.1106"></a>
-<span class="sourceLineNo">1107</span>  }<a name="line.1107"></a>
-<span class="sourceLineNo">1108</span><a name="line.1108"></a>
-<span class="sourceLineNo">1109</span>  @Override<a name="line.1109"></a>
-<span class="sourceLineNo">1110</span>  public ServerManager getServerManager() {<a name="line.1110"></a>
-<span class="sourceLineNo">1111</span>    return this.serverManager;<a name="line.1111"></a>
-<span class="sourceLineNo">1112</span>  }<a name="line.1112"></a>
-<span class="sourceLineNo">1113</span><a name="line.1113"></a>
-<span class="sourceLineNo">1114</span>  @Override<a name="line.1114"></a>
-<span class="sourceLineNo">1115</span>  public MasterFileSystem getMasterFileSystem() {<a name="line.1115"></a>
-<span class="sourceLineNo">1116</span>    return this.fileSystemManager;<a name="line.1116"></a>
-<span class="sourceLineNo">1117</span>  }<a name="line.1117"></a>
-<span class="sourceLineNo">1118</span><a name="line.1118"></a>
-<span class="sourceLineNo">1119</span>  @Override<a name="line.1119"></a>
-<span class="sourceLineNo">1120</span>  public MasterWalManager getMasterWalManager() {<a name="line.1120"></a>
-<span class="sourceLineNo">1121</span>    return this.walManager;<a name="line.1121"></a>
-<span class="sourceLineNo">1122</span>  }<a name="line.1122"></a>
-<span class="sourceLineNo">1123</span><a name="line.1123"></a>
-<span class="sourceLineNo">1124</span>  @Override<a name="line.1124"></a>
-<span class="sourceLineNo">1125</span>  public TableStateManager getTableStateManager() {<a name="line.1125"></a>
-<span class="sourceLineNo">1126</span>    return tableStateManager;<a name="line.1126"></a>
-<span class="sourceLineNo">1127</span>  }<a name="line.1127"></a>
-<span class="sourceLineNo">1128</span><a name="line.1128"></a>
-<span class="sourceLineNo">1129</span>  /*<a name="line.1129"></a>
-<span class="sourceLineNo">1130</span>   * Start up all services. If any of these threads gets an unhandled exception<a name="line.1130"></a>
-<span class="sourceLineNo">1131</span>   * then they just die with a logged message.  This should be fine because<a name="line.1131"></a>
-<span class="sourceLineNo">1132</span>   * in general, we do not expect the master to get such unhandled exceptions<a name="line.1132"></a>
-<span class="sourceLineNo">1133</span>   *  as OOMEs; it should be lightly loaded. See what HRegionServer does if<a name="line.1133"></a>
-<span class="sourceLineNo">1134</span>   *  need to install an unexpected exception handler.<a name="line.1134"></a>
-<span class="sourceLineNo">1135</span>   */<a name="line.1135"></a>
-<span class="sourceLineNo">1136</span>  private void startServiceThreads() throws IOException{<a name="line.1136"></a>
-<span class="sourceLineNo">1137</span>   // Start the executor service pools<a name="line.1137"></a>
-<span class="sourceLineNo">1138</span>   this.executorService.startExecutorService(ExecutorType.MASTER_OPEN_REGION,<a name="line.1138"></a>
-<span class="sourceLineNo">1139</span>      conf.getInt("hbase.master.executor.openregion.threads", 5));<a name="line.1139"></a>
-<span class="sourceLineNo">1140</span>   this.executorService.startExecutorService(ExecutorType.MASTER_CLOSE_REGION,<a name="line.1140"></a>
-<span class="sourceLineNo">1141</span>      conf.getInt("hbase.master.executor.closeregion.threads", 5));<a name="line.1141"></a>
-<span class="sourceLineNo">1142</span>   this.executorService.startExecutorService(ExecutorType.MASTER_SERVER_OPERATIONS,<a name="line.1142"></a>
-<span class="sourceLineNo">1143</span>      conf.getInt("hbase.master.executor.serverops.threads", 5));<a name="line.1143"></a>
-<span class="sourceLineNo">1144</span>   this.executorService.startExecutorService(ExecutorType.MASTER_META_SERVER_OPERATIONS,<a name="line.1144"></a>
-<span class="sourceLineNo">1145</span>      conf.getInt("hbase.master.executor.meta.serverops.threads", 5));<a name="line.1145"></a>
-<span class="sourceLineNo">1146</span>   this.executorService.startExecutorService(ExecutorType.M_LOG_REPLAY_OPS,<a name="line.1146"></a>
-<span class="sourceLineNo">1147</span>      conf.getInt("hbase.master.executor.logreplayops.threads", 10));<a name="line.1147"></a>
-<span class="sourceLineNo">1148</span><a name="line.1148"></a>
-<span class="sourceLineNo">1149</span>   // We depend on there being only one instance of this executor running<a name="line.1149"></a>
-<span class="sourceLineNo">1150</span>   // at a time.  To do concurrency, would need fencing of enable/disable of<a name="line.1150"></a>
-<span class="sourceLineNo">1151</span>   // tables.<a name="line.1151"></a>
-<span class="sourceLineNo">1152</span>   // Any time changing this maxThreads to &gt; 1, pls see the comment at<a name="line.1152"></a>
-<span class="sourceLineNo">1153</span>   // AccessController#postCompletedCreateTableAction<a name="line.1153"></a>
-<span class="sourceLineNo">1154</span>   this.executorService.startExecutorService(ExecutorType.MASTER_TABLE_OPERATIONS, 1);<a name="line.1154"></a>
-<span class="sourceLineNo">1155</span>   startProcedureExecutor();<a name="line.1155"></a>
-<span class="sourceLineNo">1156</span><a name="line.1156"></a>
-<span class="sourceLineNo">1157</span>   // Start log cleaner thread<a name="line.1157"></a>
-<span class="sourceLineNo">1158</span>   int cleanerInterval = conf.getInt("hbase.master.cleaner.interval", 60 * 1000);<a name="line.1158"></a>
-<span class="sourceLineNo">1159</span>   this.logCleaner =<a name="line.1159"></a>
-<span class="sourceLineNo">1160</span>      new LogCleaner(cleanerInterval,<a name="line.1160"></a>
-<span class="sourceLineNo">1161</span>         this, conf, getMasterWalManager().getFileSystem(),<a name="line.1161"></a>
-<span class="sourceLineNo">1162</span>         getMasterWalManager().getOldLogDir());<a name="line.1162"></a>
-<span class="sourceLineNo">1163</span>    getChoreService().scheduleChore(logCleaner);<a name="line.1163"></a>
-<span class="sourceLineNo">1164</span><a name="line.1164"></a>
-<span class="sourceLineNo">1165</span>   //start the hfile archive cleaner thread<a name="line.1165"></a>
-<span class="sourceLineNo">1166</span>    Path archiveDir = HFileArchiveUtil.getArchivePath(conf);<a name="line.1166"></a>
-<span class="sourceLineNo">1167</span>    Map&lt;String, Object&gt; params = new HashMap&lt;&gt;();<a name="line.1167"></a>
-<span class="sourceLineNo">1168</span>    params.put(MASTER, this);<a name="line.1168"></a>
-<span class="sourceLineNo">1169</span>    this.hfileCleaner = new HFileCleaner(cleanerInterval, this, conf, getMasterFileSystem()<a name="line.1169"></a>
-<span class="sourceLineNo">1170</span>        .getFileSystem(), archiveDir, params);<a name="line.1170"></a>
-<span class="sourceLineNo">1171</span>    getChoreService().scheduleChore(hfileCleaner);<a name="line.1171"></a>
-<span class="sourceLineNo">1172</span>    serviceStarted = true;<a name="line.1172"></a>
-<span class="sourceLineNo">1173</span>    if (LOG.isTraceEnabled()) {<a name="line.1173"></a>
-<span class="sourceLineNo">1174</span>      LOG.trace("Started service threads");<a name="line.1174"></a>
-<span class="sourceLineNo">1175</span>    }<a name="line.1175"></a>
-<span class="sourceLineNo">1176</span>    replicationMetaCleaner = new ReplicationMetaCleaner(this, this, cleanerInterval);<a name="line.1176"></a>
-<span class="sourceLineNo">1177</span>    getChoreService().scheduleChore(replicationMetaCleaner);<a name="line.1177"></a>
-<span class="sourceLineNo">1178</span>  }<a name="line.1178"></a>
-<span class="sourceLineNo">1179</span><a name="line.1179"></a>
-<span class="sourceLineNo">1180</span>  @Override<a name="line.1180"></a>
-<span class="sourceLineNo">1181</span>  protected void stopServiceThreads() {<a name="line.1181"></a>
-<span class="sourceLineNo">1182</span>    if (masterJettyServer != null) {<a name="line.1182"></a>
-<span class="sourceLineNo">1183</span>      LOG.info("Stopping master jetty server");<a name="line.1183"></a>
-<span class="sourceLineNo">1184</span>      try {<a name="line.1184"></a>
-<span class="sourceLineNo">1185</span>        masterJettyServer.stop();<a name="line.1185"></a>
-<span class="sourceLineNo">1186</span>      } catch (Exception e) {<a name="line.1186"></a>
-<span class="sourceLineNo">1187</span>        LOG.error("Failed to stop master jetty server", e);<a name="line.1187"></a>
-<span class="sourceLineNo">1188</span>      }<a name="line.1188"></a>
-<span class="sourceLineNo">1189</span>    }<a name="line.1189"></a>
-<span class="sourceLineNo">1190</span>    super.stopServiceThreads();<a name="line.1190"></a>
-<span class="sourceLineNo">1191</span>    stopChores();<a name="line.1191"></a>
-<span class="sourceLineNo">1192</span><a name="line.1192"></a>
-<span class="sourceLineNo">1193</span>    if (LOG.isDebugEnabled()) {<a name="line.1193"></a>
-<span class="sourceLineNo">1194</span>      LOG.debug("Stopping service threads");<a name="line.1194"></a>
-<span class="sourceLineNo">1195</span>    }<a name="line.1195"></a>
-<span class="sourceLineNo">1196</span><a name="line.1196"></a>
-<span class="sourceLineNo">1197</span>    // Clean up and close up shop<a name="line.1197"></a>
-<span class="sourceLineNo">1198</span>    if (this.logCleaner != null) this.logCleaner.cancel(true);<a name="line.1198"></a>
-<span class="sourceLineNo">1199</span>    if (this.hfileCleaner != null) this.hfileCleaner.cancel(true);<a name="line.1199"></a>
-<span class="sourceLineNo">1200</span>    if (this.replicationMetaCleaner != null) this.replicationMetaCleaner.cancel(true);<a name="line.1200"></a>
-<span class="sourceLineNo">1201</span>    if (this.quotaManager != null) this.quotaManager.stop();<a name="line.1201"></a>
+<span class="sourceLineNo">1093</span>  @Override<a name="line.1093"></a>
+<span class="sourceLineNo">1094</span>  public TableDescriptors getTableDescriptors() {<a name="line.1094"></a>
+<span class="sourceLineNo">1095</span>    return this.tableDescriptors;<a name="line.1095"></a>
+<span class="sourceLineNo">1096</span>  }<a name="line.1096"></a>
+<span class="sourceLineNo">1097</span><a name="line.1097"></a>
+<span class="sourceLineNo">1098</span>  @Override<a name="line.1098"></a>
+<span class="sourceLineNo">1099</span>  public ServerManager getServerManager() {<a name="line.1099"></a>
+<span class="sourceLineNo">1100</span>    return this.serverManager;<a name="line.1100"></a>
+<span class="sourceLineNo">1101</span>  }<a name="line.1101"></a>
+<span class="sourceLineNo">1102</span><a name="line.1102"></a>
+<span class="sourceLineNo">1103</span>  @Override<a name="line.1103"></a>
+<span class="sourceLineNo">1104</span>  public MasterFileSystem getMasterFileSystem() {<a name="line.1104"></a>
+<span class="sourceLineNo">1105</span>    return this.fileSystemManager;<a name="line.1105"></a>
+<span class="sourceLineNo">1106</span>  }<a name="line.1106"></a>
+<span class="sourceLineNo">1107</span><a name="line.1107"></a>
+<span class="sourceLineNo">1108</span>  @Override<a name="line.1108"></a>
+<span class="sourceLineNo">1109</span>  public MasterWalManager getMasterWalManager() {<a name="line.1109"></a>
+<span class="sourceLineNo">1110</span>    return this.walManager;<a name="line.1110"></a>
+<span class="sourceLineNo">1111</span>  }<a name="line.1111"></a>
+<span class="sourceLineNo">1112</span><a name="line.1112"></a>
+<span class="sourceLineNo">1113</span>  @Override<a name="line.1113"></a>
+<span class="sourceLineNo">1114</span>  public TableStateManager getTableStateManager() {<a name="line.1114"></a>
+<span class="sourceLineNo">1115</span>    return tableStateManager;<a name="line.1115"></a>
+<span class="sourceLineNo">1116</span>  }<a name="line.1116"></a>
+<span class="sourceLineNo">1117</span><a name="line.1117"></a>
+<span class="sourceLineNo">1118</span>  /*<a name="line.1118"></a>
+<span class="sourceLineNo">1119</span>   * Start up all services. If any of these threads gets an unhandled exception<a name="line.1119"></a>
+<span class="sourceLineNo">1120</span>   * then they just die with a logged message.  This should be fine because<a name="line.1120"></a>
+<span class="sourceLineNo">1121</span>   * in general, we do not expect the master to get such unhandled exceptions<a name="line.1121"></a>
+<span class="sourceLineNo">1122</span>   *  as OOMEs; it should be lightly loaded. See what HRegionServer does if<a name="line.1122"></a>
+<span class="sourceLineNo">1123</span>   *  need to install an unexpected exception handler.<a name="line.1123"></a>
+<span class="sourceLineNo">1124</span>   */<a name="line.1124"></a>
+<span class="sourceLineNo">1125</span>  private void startServiceThreads() throws IOException{<a name="line.1125"></a>
+<span class="sourceLineNo">1126</span>   // Start the executor service pools<a name="line.1126"></a>
+<span class="sourceLineNo">1127</span>   this.executorService.startExecutorService(ExecutorType.MASTER_OPEN_REGION,<a name="line.1127"></a>
+<span class="sourceLineNo">1128</span>      conf.getInt("hbase.master.executor.openregion.threads", 5));<a name="line.1128"></a>
+<span class="sourceLineNo">1129</span>   this.executorService.startExecutorService(ExecutorType.MASTER_CLOSE_REGION,<a name="line.1129"></a>
+<span class="sourceLineNo">1130</span>      conf.getInt("hbase.master.executor.closeregion.threads", 5));<a name="line.1130"></a>
+<span class="sourceLineNo">1131</span>   this.executorService.startExecutorService(ExecutorType.MASTER_SERVER_OPERATIONS,<a name="line.1131"></a>
+<span class="sourceLineNo">1132</span>      conf.getInt("hbase.master.executor.serverops.threads", 5));<a name="line.1132"></a>
+<span class="sourceLineNo">1133</span>   this.executorService.startExecutorService(ExecutorType.MASTER_META_SERVER_OPERATIONS,<a name="line.1133"></a>
+<span class="sourceLineNo">1134</span>      conf.getInt("hbase.master.executor.meta.serverops.threads", 5));<a name="line.1134"></a>
+<span class="sourceLineNo">1135</span>   this.executorService.startExecutorService(ExecutorType.M_LOG_REPLAY_OPS,<a name="line.1135"></a>
+<span class="sourceLineNo">1136</span>      conf.getInt("hbase.master.executor.logreplayops.threads", 10));<a name="line.1136"></a>
+<span class="sourceLineNo">1137</span><a name="line.1137"></a>
+<span class="sourceLineNo">1138</span>   // We depend on there being only one instance of this executor running<a name="line.1138"></a>
+<span class="sourceLineNo">1139</span>   // at a time.  To do concurrency, would need fencing of enable/disable of<a name="line.1139"></a>
+<span class="sourceLineNo">1140</span>   // tables.<a name="line.1140"></a>
+<span class="sourceLineNo">1141</span>   // Any time changing this maxThreads to &gt; 1, pls see the comment at<a name="line.1141"></a>
+<span class="sourceLineNo">1142</span>   // AccessController#postCompletedCreateTableAction<a name="line.1142"></a>
+<span class="sourceLineNo">1143</span>   this.executorService.startExecutorService(ExecutorType.MASTER_TABLE_OPERATIONS, 1);<a name="line.1143"></a>
+<span class="sourceLineNo">1144</span>   startProcedureExecutor();<a name="line.1144"></a>
+<span class="sourceLineNo">1145</span><a name="line.1145"></a>
+<span class="sourceLineNo">1146</span>   // Start log cleaner thread<a name="line.1146"></a>
+<span class="sourceLineNo">1147</span>   int cleanerInterval = conf.getInt("hbase.master.cleaner.interval", 60 * 1000);<a name="line.1147"></a>
+<span class="sourceLineNo">1148</span>   this.logCleaner =<a name="line.1148"></a>
+<span class="sourceLineNo">1149</span>      new LogCleaner(cleanerInterval,<a name="line.1149"></a>
+<span class="sourceLineNo">1150</span>         this, conf, getMasterWalManager().getFileSystem(),<a name="line.1150"></a>
+<span class="sourceLineNo">1151</span>         getMasterWalManager().getOldLogDir());<a name="line.1151"></a>
+<span class="sourceLineNo">1152</span>    getChoreService().scheduleChore(logCleaner);<a name="line.1152"></a>
+<span class="sourceLineNo">1153</span><a name="line.1153"></a>
+<span class="sourceLineNo">1154</span>   //start the hfile archive cleaner thread<a name="line.1154"></a>
+<span class="sourceLineNo">1155</span>    Path archiveDir = HFileArchiveUtil.getArchivePath(conf);<a name="line.1155"></a>
+<span class="sourceLineNo">1156</span>    Map&lt;String, Object&gt; params = new HashMap&lt;&gt;();<a name="line.1156"></a>
+<span class="sourceLineNo">1157</span>    params.put(MASTER, this);<a name="line.1157"></a>
+<span class="sourceLineNo">1158</span>    this.hfileCleaner = new HFileCleaner(cleanerInterval, this, conf, getMasterFileSystem()<a name="line.1158"></a>
+<span class="sourceLineNo">1159</span>        .getFileSystem(), archiveDir, params);<a name="line.1159"></a>
+<span class="sourceLineNo">1160</span>    getChoreService().scheduleChore(hfileCleaner);<a name="line.1160"></a>
+<span class="sourceLineNo">1161</span>    serviceStarted = true;<a name="line.1161"></a>
+<span class="sourceLineNo">1162</span>    if (LOG.isTraceEnabled()) {<a name="line.1162"></a>
+<span class="sourceLineNo">1163</span>      LOG.trace("Started service threads");<a name="line.1163"></a>
+<span class="sourceLineNo">1164</span>    }<a name="line.1164"></a>
+<span class="sourceLineNo">1165</span>    replicationMetaCleaner = new ReplicationMetaCleaner(this, this, cleanerInterval);<a name="line.1165"></a>
+<span class="sourceLineNo">1166</span>    getChoreService().scheduleChore(replicationMetaCleaner);<a name="line.1166"></a>
+<span class="sourceLineNo">1167</span>  }<a name="line.1167"></a>
+<span class="sourceLineNo">1168</span><a name="line.1168"></a>
+<span class="sourceLineNo">1169</span>  @Override<a name="line.1169"></a>
+<span class="sourceLineNo">1170</span>  protected void stopServiceThreads() {<a name="line.1170"></a>
+<span class="sourceLineNo">1171</span>    if (masterJettyServer != null) {<a name="line.1171"></a>
+<span class="sourceLineNo">1172</span>      LOG.info("Stopping master jetty server");<a name="line.1172"></a>
+<span class="sourceLineNo">1173</span>      try {<a name="line.1173"></a>
+<span class="sourceLineNo">1174</span>        masterJettyServer.stop();<a name="line.1174"></a>
+<span class="sourceLineNo">1175</span>      } catch (Exception e) {<a name="line.1175"></a>
+<span class="sourceLineNo">1176</span>        LOG.error("Failed to stop master jetty server", e);<a name="line.1176"></a>
+<span class="sourceLineNo">1177</span>      }<a name="line.1177"></a>
+<span class="sourceLineNo">1178</span>    }<a name="line.1178"></a>
+<span class="sourceLineNo">1179</span>    super.stopServiceThreads();<a name="line.1179"></a>
+<span class="sourceLineNo">1180</span>    stopChores();<a name="line.1180"></a>
+<span class="sourceLineNo">1181</span><a name="line.1181"></a>
+<span class="sourceLineNo">1182</span>    if (LOG.isDebugEnabled()) {<a name="line.1182"></a>
+<span class="sourceLineNo">1183</span>      LOG.debug("Stopping service threads");<a name="line.1183"></a>
+<span class="sourceLineNo">1184</span>    }<a name="line.1184"></a>
+<span class="sourceLineNo">1185</span><a name="line.1185"></a>
+<span class="sourceLineNo">1186</span>    // Clean up and close up shop<a name="line.1186"></a>
+<span class="sourceLineNo">1187</span>    if (this.logCleaner != null) this.logCleaner.cancel(true);<a name="line.1187"></a>
+<span class="sourceLineNo">1188</span>    if (this.hfileCleaner != null) this.hfileCleaner.cancel(true);<a name="line.1188"></a>
+<span class="sourceLineNo">1189</span>    if (this.replicationMetaCleaner != null) this.replicationMetaCleaner.cancel(true);<a name="line.1189"></a>
+<span class="sourceLineNo">1190</span>    if (this.quotaManager != null) this.quotaManager.stop();<a name="line.1190"></a>
+<span class="sourceLineNo">1191</span><a name="line.1191"></a>
+<span class="sourceLineNo">1192</span>    if (this.activeMasterManager != null) this.activeMasterManager.stop();<a name="line.1192"></a>
+<span class="sourceLineNo">1193</span>    if (this.serverManager != null) this.serverManager.stop();<a name="line.1193"></a>
+<span class="sourceLineNo">1194</span>    if (this.assignmentManager != null) this.assignmentManager.stop();<a name="line.1194"></a>
+<span class="sourceLineNo">1195</span><a name="line.1195"></a>
+<span class="sourceLineNo">1196</span>    stopProcedureExecutor();<a name="line.1196"></a>
+<span class="sourceLineNo">1197</span><a name="line.1197"></a>
+<span class="sourceLineNo">1198</span>    if (this.walManager != null) this.walManager.stop();<a name="line.1198"></a>
+<span class="sourceLineNo">1199</span>    if (this.fileSystemManager != null) this.fileSystemManager.stop();<a name="line.1199"></a>
+<span class="sourceLineNo">1200</span>    if (this.mpmHost != null) this.mpmHost.stop("server shutting down.");<a name="line.1200"></a>
+<span class="sourceLineNo">1201</span>  }<a name="line.1201"></a>
 <span class="sourceLineNo">1202</span><a name="line.1202"></a>
-<span class="sourceLineNo">1203</span>    if (this.activeMasterManager != null) this.activeMasterManager.stop();<a name="line.1203"></a>
-<span class="sourceLineNo">1204</span>    if (this.serverManager != null) this.serverManager.stop();<a name="line.1204"></a>
-<span class="sourceLineNo">1205</span>    if (this.assignmentManager != null) this.assignmentManager.stop();<a name="line.1205"></a>
-<span class="sourceLineNo">1206</span><a name="line.1206"></a>
-<span class="sourceLineNo">1207</span>    stopProcedureExecutor();<a name="line.1207"></a>
-<span class="sourceLineNo">1208</span><a name="line.1208"></a>
-<span class="sourceLineNo">1209</span>    if (this.walManager != null) this.walManager.stop();<a name="line.1209"></a>
-<span class="sourceLineNo">1210</span>    if (this.fileSystemManager != null) this.fileSystemManager.stop();<a name="line.1210"></a>
-<span class="sourceLineNo">1211</span>    if (this.mpmHost != null) this.mpmHost.stop("server shutting down.");<a name="line.1211"></a>
-<span class="sourceLineNo">1212</span>  }<a name="line.1212"></a>
-<span class="sourceLineNo">1213</span><a name="line.1213"></a>
-<span class="sourceLineNo">1214</span>  private void startProcedureExecutor() throws IOException {<a name="line.1214"></a>
-<span class="sourceLineNo">1215</span>    final MasterProcedureEnv procEnv = new MasterProcedureEnv(this);<a name="line.1215"></a>
-<span class="sourceLineNo">1216</span><a name="line.1216"></a>
-<span class="sourceLineNo">1217</span>    procedureStore = new WALProcedureStore(conf,<a name="line.1217"></a>
-<span class="sourceLineNo">1218</span>        new MasterProcedureEnv.WALStoreLeaseRecovery(this));<a name="line.1218"></a>
-<span class="sourceLineNo">1219</span>    procedureStore.registerListener(new MasterProcedureEnv.MasterProcedureStoreListener(this));<a name="line.1219"></a>
-<span class="sourceLineNo">1220</span>    MasterProcedureScheduler procedureScheduler = procEnv.getProcedureScheduler();<a name="line.1220"></a>
-<span class="sourceLineNo">1221</span>    procedureExecutor = new ProcedureExecutor&lt;&gt;(conf, procEnv, procedureStore, procedureScheduler);<a name="line.1221"></a>
-<span class="sourceLineNo">1222</span>    configurationManager.registerObserver(procEnv);<a name="line.1222"></a>
+<span class="sourceLineNo">1203</span>  private void startProcedureExecutor() throws IOException {<a name="line.1203"></a>
+<span class="sourceLineNo">1204</span>    final MasterProcedureEnv procEnv = new MasterProcedureEnv(this);<a name="line.1204"></a>
+<span class="sourceLineNo">1205</span><a name="line.1205"></a>
+<span class="sourceLineNo">1206</span>    procedureStore = new WALProcedureStore(conf,<a name="line.1206"></a>
+<span class="sourceLineNo">1207</span>        new MasterProcedureEnv.WALStoreLeaseRecovery(this));<a name="line.1207"></a>
+<span class="sourceLineNo">1208</span>    procedureStore.registerListener(new MasterProcedureEnv.MasterProcedureStoreListener(this));<a name="line.1208"></a>
+<span class="sourceLineNo">1209</span>    MasterProcedureScheduler procedureScheduler = procEnv.getProcedureScheduler();<a name="line.1209"></a>
+<span class="sourceLineNo">1210</span>    procedureExecutor = new ProcedureExecutor&lt;&gt;(conf, procEnv, procedureStore, procedureScheduler);<a name="line.1210"></a>
+<span class="sourceLineNo">1211</span>    configurationManager.registerObserver(procEnv);<a name="line.1211"></a>
+<span class="sourceLineNo">1212</span><a name="line.1212"></a>
+<span class="sourceLineNo">1213</span>    final int numThreads = conf.getInt(MasterProcedureConstants.MASTER_PROCEDURE_THREADS,<a name="line.1213"></a>
+<span class="sourceLineNo">1214</span>        Math.max(Runtime.getRuntime().availableProcessors(),<a name="line.1214"></a>
+<span class="sourceLineNo">1215</span>          MasterProcedureConstants.DEFAULT_MIN_MASTER_PROCEDURE_THREADS));<a name="line.1215"></a>
+<span class="sourceLineNo">1216</span>    final boolean abortOnCorruption = conf.getBoolean(<a name="line.1216"></a>
+<span class="sourceLineNo">1217</span>        MasterProcedureConstants.EXECUTOR_ABORT_ON_CORRUPTION,<a name="line.1217"></a>
+<span class="sourceLineNo">1218</span>        MasterProcedureConstants.DEFAULT_EXECUTOR_ABORT_ON_CORRUPTION);<a name="line.1218"></a>
+<span class="sourceLineNo">1219</span>    procedureStore.start(numThreads);<a name="line.1219"></a>
+<span class="sourceLineNo">1220</span>    procedureExecutor.start(numThreads, abortOnCorruption);<a name="line.1220"></a>
+<span class="sourceLineNo">1221</span>    procEnv.getRemoteDispatcher().start();<a name="line.1221"></a>
+<span class="sourceLineNo">1222</span>  }<a name="line.1222"></a>
 <span class="sourceLineNo">1223</span><a name="line.1223"></a>
-<span class="sourceLineNo">1224</span>    final int numThreads = conf.getInt(MasterProcedureConstants.MASTER_PROCEDURE_THREADS,<a name="line.1224"></a>
-<span class="sourceLineNo">1225</span>        Math.max(Runtime.getRuntime().availableProcessors(),<a name="line.1225"></a>
-<span class="sourceLineNo">1226</span>          MasterProcedureConstants.DEFAULT_MIN_MASTER_PROCEDURE_THREADS));<a name="line.1226"></a>
-<span class="sourceLineNo">1227</span>    final boolean abortOnCorruption = conf.getBoolean(<a name="line.1227"></a>
-<span class="sourceLineNo">1228</span>        MasterProcedureConstants.EXECUTOR_ABORT_ON_CORRUPTION,<a name="line.1228"></a>
-<span class="sourceLineNo">1229</span>        MasterProcedureConstants.DEFAULT_EXECUTOR_ABORT_ON_CORRUPTION);<a name="line.1229"></a>
-<span class="sourceLineNo">1230</span>    procedureStore.start(numThreads);<a name="line.1230"></a>
-<span class="sourceLineNo">1231</span>    procedureExecutor.start(numThreads, abortOnCorruption);<a name="line.1231"></a>
-<span class="sourceLineNo">1232</span>    procEnv.getRemoteDispatcher().start();<a name="line.1232"></a>
-<span class="sourceLineNo">1233</span>  }<a name="line.1233"></a>
-<span class="sourceLineNo">1234</span><a name="line.1234"></a>
-<span class="sourceLineNo">1235</span>  private void stopProcedureExecutor() {<a name="line.1235"></a>
-<span class="sourceLineNo">1236</span>    if (procedureExecutor != null) {<a name="line.1236"></a>
-<span class="sourceLineNo">1237</span>      configurationManager.deregisterObserver(procedureExecutor.getEnvironment());<a name="line.1237"></a>
-<span class="sourceLineNo">1238</span>      procedureExecutor.getEnvironment().getRemoteDispatcher().stop();<a name="line.1238"></a>
-<span class="sourceLineNo">1239</span>      procedureExecutor.stop();<a name="line.1239"></a>
-<span class="sourceLineNo">1240</span>      procedureExecutor.join();<a name="line.1240"></a>
-<span class="sourceLineNo">1241</span>      procedureExecutor = null;<a name="line.1241"></a>
+<span class="sourceLineNo">1224</span>  private void stopProcedureExecutor() {<a name="line.1224"></a>
+<span class="sourceLineNo">1225</span>    if (procedureExecutor != null) {<a name="line.1225"></a>
+<span class="sourceLineNo">1226</span>      configurationManager.deregisterObserver(procedureExecutor.getEnvironment());<a name="line.1226"></a>
+<span class="sourceLineNo">1227</span>      procedureExecutor.getEnvironment().getRemoteDispatcher().stop();<a name="line.1227"></a>
+<span class="sourceLineNo">1228</span>      procedureExecutor.stop();<a name="line.1228"></a>
+<span class="sourceLineNo">1229</span>      procedureExecutor.join();<a name="line.1229"></a>
+<span class="sourceLineNo">1230</span>      procedureExecutor = null;<a name="line.1230"></a>
+<span class="sourceLineNo">1231</span>    }<a name="line.1231"></a>
+<span class="sourceLineNo">1232</span><a name="line.1232"></a>
+<span class="sourceLineNo">1233</span>    if (procedureStore != null) {<a name="line.1233"></a>
+<span class="sourceLineNo">1234</span>      procedureStore.stop(isAborted());<a name="line.1234"></a>
+<span class="sourceLineNo">1235</span>      procedureStore = null;<a name="line.1235"></a>
+<span class="sourceLineNo">1236</span>    }<a name="line.1236"></a>
+<span class="sourceLineNo">1237</span>  }<a name="line.1237"></a>
+<span class="sourceLineNo">1238</span><a name="line.1238"></a>
+<span class="sourceLineNo">1239</span>  private void stopChores() {<a name="line.1239"></a>
+<span class="sourceLineNo">1240</span>    if (this.expiredMobFileCleanerChore != null) {<a name="line.1240"></a>
+<span class="sourceLineNo">1241</span>      this.expiredMobFileCleanerChore.cancel(true);<a name="line.1241"></a>
 <span class="sourceLineNo">1242</span>    }<a name="line.1242"></a>
-<span class="sourceLineNo">1243</span><a name="line.1243"></a>
-<span class="sourceLineNo">1244</span>    if (procedureStore != null) {<a name="line.1244"></a>
-<span class="sourceLineNo">1245</span>      procedureStore.stop(isAborted());<a name="line.1245"></a>
-<span class="sourceLineNo">1246</span>      procedureStore = null;<a name="line.1246"></a>
-<span class="sourceLineNo">1247</span>    }<a name="line.1247"></a>
-<span class="sourceLineNo">1248</span>  }<a name="line.1248"></a>
-<span class="sourceLineNo">1249</span><a name="line.1249"></a>
-<span class="sourceLineNo">1250</span>  private void stopChores() {<a name="line.1250"></a>
-<span class="sourceLineNo">1251</span>    if (this.expiredMobFileCleanerChore != null) {<a name="line.1251"></a>
-<span class="sourceLineNo">1252</span>      this.expiredMobFileCleanerChore.cancel(true);<a name="line.1252"></a>
-<span class="sourceLineNo">1253</span>    }<a name="line.1253"></a>
-<span class="sourceLineNo">1254</span>    if (this.mobCompactChore != null) {<a name="line.1254"></a>
-<span class="sourceLineNo">1255</span>      this.mobCompactChore.cancel(true);<a name="line.1255"></a>
-<span class="sourceLineNo">1256</span>    }<a name="line.1256"></a>
-<span class="sourceLineNo">1257</span>    if (this.balancerChore != null) {<a name="line.1257"></a>
-<span class="sourceLineNo">1258</span>      this.balancerChore.cancel(true);<a name="line.1258"></a>
-<span class="sourceLineNo">1259</span>    }<a name="line.1259"></a>
-<span class="sourceLineNo">1260</span>    if (this.normalizerChore != null) {<a name="line.1260"></a>
-<span class="sourceLineNo">1261</span>      this.normalizerChore.cancel(true);<a name="line.1261"></a>
-<span class="sourceLineNo">1262</span>    }<a name="line.1262"></a>
-<span class="sourceLineNo">1263</span>    if (this.clusterStatusChore != null) {<a name="line.1263"></a>
-<span class="sourceLineNo">1264</span>      this.clusterStatusChore.cancel(true);<a name="line.1264"></a>
-<span class="sourceLineNo">1265</span>    }<a name="line.1265"></a>
-<span class="sourceLineNo">1266</span>    if (this.catalogJanitorChore != null) {<a name="line.1266"></a>
-<span class="sourceLineNo">1267</span>      this.catalogJanitorChore.cancel(true);<a name="line.1267"></a>
-<span class="sourceLineNo">1268</span>    }<a name="line.1268"></a>
-<span class="sourceLineNo">1269</span>    if (this.clusterStatusPublisherChore != null){<a name="line.1269"></a>
-<span class="sourceLineNo">1270</span>      clusterStatusPublisherChore.cancel(true);<a name="line.1270"></a>
-<span class="sourceLineNo">1271</span>    }<a name="line.1271"></a>
-<span class="sourceLineNo">1272</span>    if (this.mobCompactThread != null) {<a name="line.1272"></a>
-<span class="sourceLineNo">1273</span>      this.mobCompactThread.close();<a name="line.1273"></a>
-<span class="sourceLineNo">1274</span>    }<a name="line.1274"></a>
-<span class="sourceLineNo">1275</span><a name="line.1275"></a>
-<span class="sourceLineNo">1276</span>    if (this.quotaObserverChore != null) {<a name="line.1276"></a>
-<span class="sourceLineNo">1277</span>      quotaObserverChore.cancel();<a name="line.1277"></a>
-<span class="sourceLineNo">1278</span>    }<a name="line.1278"></a>
-<span class="sourceLineNo">1279</span>    if (this.snapshotQuotaChore != null) {<a name="line.1279"></a>
-<span class="sourceLineNo">1280</span>      snapshotQuotaChore.cancel();<a name="line.1280"></a>
-<span class="sourceLineNo">1281</span>    }<a name="line.1281"></a>
-<span class="sourceLineNo">1282</span>  }<a name="line.1282"></a>
-<span class="sourceLineNo">1283</span><a name="line.1283"></a>
-<span class="sourceLineNo">1284</span>  /**<a name="line.1284"></a>
-<span class="sourceLineNo">1285</span>   * @return Get remote side's InetAddress<a name="line.1285"></a>
-<span class="sourceLineNo">1286</span>   */<a name="line.1286"></a>
-<span class="sourceLineNo">1287</span>  InetAddress getRemoteInetAddress(final int port,<a name="line.1287"></a>
-<span class="sourceLineNo">1288</span>      final long serverStartCode) throws UnknownHostException {<a name="line.1288"></a>
-<span class="sourceLineNo">1289</span>    // Do it out here in its own little method so can fake an address when<a name="line.1289"></a>
-<span class="sourceLineNo">1290</span>    // mocking up in tests.<a name="line.1290"></a>
-<span class="sourceLineNo">1291</span>    InetAddress ia = RpcServer.getRemoteIp();<a name="line.1291"></a>
+<span class="sourceLineNo">1243</span>    if (this.mobCompactChore != null) {<a name="line.1243"></a>
+<span class="sourceLineNo">1244</span>      this.mobCompactChore.cancel(true);<a name="line.1244"></a>
+<span class="sourceLineNo">1245</span>    }<a name="line.1245"></a>
+<span class="sourceLineNo">1246</span>    if (this.balancerChore != null) {<a name="line.1246"></a>
+<span class="sourceLineNo">1247</span>      this.balancerChore.cancel(true);<a name="line.1247"></a>
+<span class="sourceLineNo">1248</span>    }<a name="line.1248"></a>
+<span class="sourceLineNo">1249</span>    if (this.normalizerChore != null) {<a name="line.1249"></a>
+<span class="sourceLineNo">1250</span>      this.normalizerChore.cancel(true);<a name="line.1250"></a>
+<span class="sourceLineNo">1251</span>    }<a name="line.1251"></a>
+<span class="sourceLineNo">1252</span>    if (this.clusterStatusChore != null) {<a name="line.1252"></a>
+<span class="sourceLineNo">1253</span>      this.clusterStatusChore.cancel(true);<a name="line.1253"></a>
+<span class="sourceLineNo">1254</span>    }<a name="line.1254"></a>
+<span class="sourceLineNo">1255</span>    if (this.catalogJanitorChore != null) {<a name="line.1255"></a>
+<span class="sourceLineNo">1256</span>      this.catalogJanitorChore.cancel(true);<a name="line.1256"></a>
+<span class="sourceLineNo">1257</span>    }<a name="line.1257"></a>
+<span class="sourceLineNo">1258</span>    if (this.clusterStatusPublisherChore != null){<a name="line.1258"></a>
+<span class="sourceLineNo">1259</span>      clusterStatusPublisherChore.cancel(true);<a name="line.1259"></a>
+<span class="sourceLineNo">1260</span>    }<a name="line.1260"></a>
+<span class="sourceLineNo">1261</span>    if (this.mobCompactThread != null) {<a name="line.1261"></a>
+<span class="sourceLineNo">1262</span>      this.mobCompactThread.close();<a name="line.1262"></a>
+<span class="sourceLineNo">1263</span>    }<a name="line.1263"></a>
+<span class="sourceLineNo">1264</span><a name="line.1264"></a>
+<span class="sourceLineNo">1265</span>    if (this.quotaObserverChore != null) {<a name="line.1265"></a>
+<span class="sourceLineNo">1266</span>      quotaObserverChore.cancel();<a name="line.1266"></a>
+<span class="sourceLineNo">1267</span>    }<a name="line.1267"></a>
+<span class="sourceLineNo">1268</span>    if (this.snapshotQuotaChore != null) {<a name="line.1268"></a>
+<span class="sourceLineNo">1269</span>      snapshotQuotaChore.cancel();<a name="line.1269"></a>
+<span class="sourceLineNo">1270</span>    }<a name="line.1270"></a>
+<span class="sourceLineNo">1271</span>  }<a name="line.1271"></a>
+<span class="sourceLineNo">1272</span><a name="line.1272"></a>
+<span class="sourceLineNo">1273</span>  /**<a name="line.1273"></a>
+<span class="sourceLineNo">1274</span>   * @return Get remote side's InetAddress<a name="line.1274"></a>
+<span class="sourceLineNo">1275</span>   */<a name="line.1275"></a>
+<span class="sourceLineNo">1276</span>  InetAddress getRemoteInetAddress(final int port,<a name="line.1276"></a>
+<span class="sourceLineNo">1277</span>      final long serverStartCode) throws UnknownHostException {<a name="line.1277"></a>
+<span class="sourceLineNo">1278</span>    // Do it out here in its own little method so can fake an address when<a name="line.1278"></a>
+<span class="sourceLineNo">1279</span>    // mocking up in tests.<a name="line.1279"></a>
+<span class="sourceLineNo">1280</span>    InetAddress ia = RpcServer.getRemoteIp();<a name="line.1280"></a>
+<span class="sourceLineNo">1281</span><a name="line.1281"></a>
+<span class="sourceLineNo">1282</span>    // The call could be from the local regionserver,<a name="line.1282"></a>
+<span class="sourceLineNo">1283</span>    // in which case, there is no remote address.<a name="line.1283"></a>
+<span class="sourceLineNo">1284</span>    if (ia == null &amp;&amp; serverStartCode == startcode) {<a name="line.1284"></a>
+<span class="sourceLineNo">1285</span>      InetSocketAddress isa = rpcServices.getSocketAddress();<a name="line.1285"></a>
+<span class="sourceLineNo">1286</span>      if (isa != null &amp;&amp; isa.getPort() == port) {<a name="line.1286"></a>
+<span class="sourceLineNo">1287</span>        ia = isa.getAddress();<a name="line.1287"></a>
+<span class="sourceLineNo">1288</span>      }<a name="line.1288"></a>
+<span class="sourceLineNo">1289</span>    }<a name="line.1289"></a>
+<span class="sourceLineNo">1290</span>    return ia;<a name="line.1290"></a>
+<span class="sourceLineNo">1291</span>  }<a name="line.1291"></a>
 <span class="sourceLineNo">12

<TRUNCATED>