[05/12] hbase-site git commit: Published site at e705cf1447dd7e44f0cc0297746a0f14135e2f2f.

[git-site-role]

http://git-wip-us.apache.org/repos/asf/hbase-site/blob/27febf40/devapidocs/src-html/org/apache/hadoop/hbase/master/assignment/AssignmentManager.html
----------------------------------------------------------------------
diff --git 
a/devapidocs/src-html/org/apache/hadoop/hbase/master/assignment/AssignmentManager.html
 
b/devapidocs/src-html/org/apache/hadoop/hbase/master/assignment/AssignmentManager.html
index 2e75659..3070994 100644
--- 
a/devapidocs/src-html/org/apache/hadoop/hbase/master/assignment/AssignmentManager.html
+++ 
b/devapidocs/src-html/org/apache/hadoop/hbase/master/assignment/AssignmentManager.html
@@ -858,1046 +858,1058 @@
 <span class="sourceLineNo">850</span><a name="line.850"></a>
 <span class="sourceLineNo">851</span>    final ServerStateNode serverNode = 
regionStates.getOrCreateServer(serverName);<a name="line.851"></a>
 <span class="sourceLineNo">852</span>    if (!reportTransition(regionNode, 
serverNode, state, seqId)) {<a name="line.852"></a>
-<span class="sourceLineNo">853</span>      // Don't log if shutting down 
cluster; during shutdown.<a name="line.853"></a>
-<span class="sourceLineNo">854</span>      LOG.warn("No matching procedure 
found for {} transition to {}", regionNode, state);<a name="line.854"></a>
-<span class="sourceLineNo">855</span>    }<a name="line.855"></a>
-<span class="sourceLineNo">856</span>  }<a name="line.856"></a>
-<span class="sourceLineNo">857</span><a name="line.857"></a>
-<span class="sourceLineNo">858</span>  // FYI: regionNode is sometimes 
synchronized by the caller but not always.<a name="line.858"></a>
-<span class="sourceLineNo">859</span>  private boolean reportTransition(final 
RegionStateNode regionNode,<a name="line.859"></a>
-<span class="sourceLineNo">860</span>      final ServerStateNode serverNode, 
final TransitionCode state, final long seqId)<a name="line.860"></a>
-<span class="sourceLineNo">861</span>      throws UnexpectedStateException {<a 
name="line.861"></a>
-<span class="sourceLineNo">862</span>    final ServerName serverName = 
serverNode.getServerName();<a name="line.862"></a>
-<span class="sourceLineNo">863</span>    synchronized (regionNode) {<a 
name="line.863"></a>
-<span class="sourceLineNo">864</span>      final RegionTransitionProcedure 
proc = regionNode.getProcedure();<a name="line.864"></a>
-<span class="sourceLineNo">865</span>      if (proc == null) return false;<a 
name="line.865"></a>
-<span class="sourceLineNo">866</span><a name="line.866"></a>
-<span class="sourceLineNo">867</span>      // 
serverNode.getReportEvent().removeProcedure(proc);<a name="line.867"></a>
-<span class="sourceLineNo">868</span>      
proc.reportTransition(master.getMasterProcedureExecutor().getEnvironment(),<a 
name="line.868"></a>
-<span class="sourceLineNo">869</span>        serverName, state, seqId);<a 
name="line.869"></a>
-<span class="sourceLineNo">870</span>    }<a name="line.870"></a>
-<span class="sourceLineNo">871</span>    return true;<a name="line.871"></a>
-<span class="sourceLineNo">872</span>  }<a name="line.872"></a>
-<span class="sourceLineNo">873</span><a name="line.873"></a>
-<span class="sourceLineNo">874</span>  private void 
updateRegionSplitTransition(final ServerName serverName, final TransitionCode 
state,<a name="line.874"></a>
-<span class="sourceLineNo">875</span>      final RegionInfo parent, final 
RegionInfo hriA, final RegionInfo hriB)<a name="line.875"></a>
-<span class="sourceLineNo">876</span>      throws IOException {<a 
name="line.876"></a>
-<span class="sourceLineNo">877</span>    checkMetaLoaded(parent);<a 
name="line.877"></a>
+<span class="sourceLineNo">853</span>      // Don't log WARN if shutting down 
cluster; during shutdown. Avoid the below messages:<a name="line.853"></a>
+<span class="sourceLineNo">854</span>      // 2018-08-13 10:45:10,551 WARN 
...AssignmentManager: No matching procedure found for<a name="line.854"></a>
+<span class="sourceLineNo">855</span>      //   rit=OPEN, 
location=ve0538.halxg.cloudera.com,16020,1533493000958,<a name="line.855"></a>
+<span class="sourceLineNo">856</span>      //   
table=IntegrationTestBigLinkedList, region=65ab289e2fc1530df65f6c3d7cde7aa5 
transition<a name="line.856"></a>
+<span class="sourceLineNo">857</span>      //   to CLOSED<a 
name="line.857"></a>
+<span class="sourceLineNo">858</span>      // These happen because on cluster 
shutdown, we currently let the RegionServers close<a name="line.858"></a>
+<span class="sourceLineNo">859</span>      // regions. This is the only time 
that region close is not run by the Master (so cluster<a name="line.859"></a>
+<span class="sourceLineNo">860</span>      // goes down fast). Consider 
changing it so Master runs all shutdowns.<a name="line.860"></a>
+<span class="sourceLineNo">861</span>      if 
(this.master.getServerManager().isClusterShutdown() &amp;&amp;<a 
name="line.861"></a>
+<span class="sourceLineNo">862</span>          
state.equals(TransitionCode.CLOSED)) {<a name="line.862"></a>
+<span class="sourceLineNo">863</span>        LOG.info("RegionServer {} {}", 
state, regionNode.getRegionInfo().getEncodedName());<a name="line.863"></a>
+<span class="sourceLineNo">864</span>      } else {<a name="line.864"></a>
+<span class="sourceLineNo">865</span>        LOG.warn("No matching procedure 
found for {} transition to {}", regionNode, state);<a name="line.865"></a>
+<span class="sourceLineNo">866</span>      }<a name="line.866"></a>
+<span class="sourceLineNo">867</span>    }<a name="line.867"></a>
+<span class="sourceLineNo">868</span>  }<a name="line.868"></a>
+<span class="sourceLineNo">869</span><a name="line.869"></a>
+<span class="sourceLineNo">870</span>  // FYI: regionNode is sometimes 
synchronized by the caller but not always.<a name="line.870"></a>
+<span class="sourceLineNo">871</span>  private boolean reportTransition(final 
RegionStateNode regionNode,<a name="line.871"></a>
+<span class="sourceLineNo">872</span>      final ServerStateNode serverNode, 
final TransitionCode state, final long seqId)<a name="line.872"></a>
+<span class="sourceLineNo">873</span>      throws UnexpectedStateException {<a 
name="line.873"></a>
+<span class="sourceLineNo">874</span>    final ServerName serverName = 
serverNode.getServerName();<a name="line.874"></a>
+<span class="sourceLineNo">875</span>    synchronized (regionNode) {<a 
name="line.875"></a>
+<span class="sourceLineNo">876</span>      final RegionTransitionProcedure 
proc = regionNode.getProcedure();<a name="line.876"></a>
+<span class="sourceLineNo">877</span>      if (proc == null) return false;<a 
name="line.877"></a>
 <span class="sourceLineNo">878</span><a name="line.878"></a>
-<span class="sourceLineNo">879</span>    if (state != 
TransitionCode.READY_TO_SPLIT) {<a name="line.879"></a>
-<span class="sourceLineNo">880</span>      throw new 
UnexpectedStateException("unsupported split regionState=" + state +<a 
name="line.880"></a>
-<span class="sourceLineNo">881</span>        " for parent region " + parent 
+<a name="line.881"></a>
-<span class="sourceLineNo">882</span>        " maybe an old RS (&lt; 2.0) had 
the operation in progress");<a name="line.882"></a>
-<span class="sourceLineNo">883</span>    }<a name="line.883"></a>
-<span class="sourceLineNo">884</span><a name="line.884"></a>
-<span class="sourceLineNo">885</span>    // sanity check on the request<a 
name="line.885"></a>
-<span class="sourceLineNo">886</span>    if (!Bytes.equals(hriA.getEndKey(), 
hriB.getStartKey())) {<a name="line.886"></a>
-<span class="sourceLineNo">887</span>      throw new 
UnsupportedOperationException(<a name="line.887"></a>
-<span class="sourceLineNo">888</span>        "unsupported split request with 
bad keys: parent=" + parent +<a name="line.888"></a>
-<span class="sourceLineNo">889</span>        " hriA=" + hriA + " hriB=" + 
hriB);<a name="line.889"></a>
-<span class="sourceLineNo">890</span>    }<a name="line.890"></a>
-<span class="sourceLineNo">891</span><a name="line.891"></a>
-<span class="sourceLineNo">892</span>    // Submit the Split procedure<a 
name="line.892"></a>
-<span class="sourceLineNo">893</span>    final byte[] splitKey = 
hriB.getStartKey();<a name="line.893"></a>
-<span class="sourceLineNo">894</span>    if (LOG.isDebugEnabled()) {<a 
name="line.894"></a>
-<span class="sourceLineNo">895</span>      LOG.debug("Split request from " + 
serverName +<a name="line.895"></a>
-<span class="sourceLineNo">896</span>          ", parent=" + parent + " 
splitKey=" + Bytes.toStringBinary(splitKey));<a name="line.896"></a>
-<span class="sourceLineNo">897</span>    }<a name="line.897"></a>
-<span class="sourceLineNo">898</span>    
master.getMasterProcedureExecutor().submitProcedure(createSplitProcedure(parent,
 splitKey));<a name="line.898"></a>
-<span class="sourceLineNo">899</span><a name="line.899"></a>
-<span class="sourceLineNo">900</span>    // If the RS is &lt; 2.0 throw an 
exception to abort the operation, we are handling the split<a 
name="line.900"></a>
-<span class="sourceLineNo">901</span>    if 
(master.getServerManager().getVersionNumber(serverName) &lt; 0x0200000) {<a 
name="line.901"></a>
-<span class="sourceLineNo">902</span>      throw new 
UnsupportedOperationException(String.format(<a name="line.902"></a>
-<span class="sourceLineNo">903</span>        "Split handled by the master: 
parent=%s hriA=%s hriB=%s", parent.getShortNameToLog(), hriA, hriB));<a 
name="line.903"></a>
-<span class="sourceLineNo">904</span>    }<a name="line.904"></a>
-<span class="sourceLineNo">905</span>  }<a name="line.905"></a>
-<span class="sourceLineNo">906</span><a name="line.906"></a>
-<span class="sourceLineNo">907</span>  private void 
updateRegionMergeTransition(final ServerName serverName, final TransitionCode 
state,<a name="line.907"></a>
-<span class="sourceLineNo">908</span>      final RegionInfo merged, final 
RegionInfo hriA, final RegionInfo hriB) throws IOException {<a 
name="line.908"></a>
-<span class="sourceLineNo">909</span>    checkMetaLoaded(merged);<a 
name="line.909"></a>
-<span class="sourceLineNo">910</span><a name="line.910"></a>
-<span class="sourceLineNo">911</span>    if (state != 
TransitionCode.READY_TO_MERGE) {<a name="line.911"></a>
-<span class="sourceLineNo">912</span>      throw new 
UnexpectedStateException("Unsupported merge regionState=" + state +<a 
name="line.912"></a>
-<span class="sourceLineNo">913</span>        " for regionA=" + hriA + " 
regionB=" + hriB + " merged=" + merged +<a name="line.913"></a>
-<span class="sourceLineNo">914</span>        " maybe an old RS (&lt; 2.0) had 
the operation in progress");<a name="line.914"></a>
-<span class="sourceLineNo">915</span>    }<a name="line.915"></a>
-<span class="sourceLineNo">916</span><a name="line.916"></a>
-<span class="sourceLineNo">917</span>    // Submit the Merge procedure<a 
name="line.917"></a>
-<span class="sourceLineNo">918</span>    if (LOG.isDebugEnabled()) {<a 
name="line.918"></a>
-<span class="sourceLineNo">919</span>      LOG.debug("Handling merge request 
from RS=" + merged + ", merged=" + merged);<a name="line.919"></a>
-<span class="sourceLineNo">920</span>    }<a name="line.920"></a>
-<span class="sourceLineNo">921</span>    
master.getMasterProcedureExecutor().submitProcedure(createMergeProcedure(hriA, 
hriB));<a name="line.921"></a>
+<span class="sourceLineNo">879</span>      // 
serverNode.getReportEvent().removeProcedure(proc);<a name="line.879"></a>
+<span class="sourceLineNo">880</span>      
proc.reportTransition(master.getMasterProcedureExecutor().getEnvironment(),<a 
name="line.880"></a>
+<span class="sourceLineNo">881</span>        serverName, state, seqId);<a 
name="line.881"></a>
+<span class="sourceLineNo">882</span>    }<a name="line.882"></a>
+<span class="sourceLineNo">883</span>    return true;<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>  private void 
updateRegionSplitTransition(final ServerName serverName, final TransitionCode 
state,<a name="line.886"></a>
+<span class="sourceLineNo">887</span>      final RegionInfo parent, final 
RegionInfo hriA, final RegionInfo hriB)<a name="line.887"></a>
+<span class="sourceLineNo">888</span>      throws IOException {<a 
name="line.888"></a>
+<span class="sourceLineNo">889</span>    checkMetaLoaded(parent);<a 
name="line.889"></a>
+<span class="sourceLineNo">890</span><a name="line.890"></a>
+<span class="sourceLineNo">891</span>    if (state != 
TransitionCode.READY_TO_SPLIT) {<a name="line.891"></a>
+<span class="sourceLineNo">892</span>      throw new 
UnexpectedStateException("unsupported split regionState=" + state +<a 
name="line.892"></a>
+<span class="sourceLineNo">893</span>        " for parent region " + parent 
+<a name="line.893"></a>
+<span class="sourceLineNo">894</span>        " maybe an old RS (&lt; 2.0) had 
the operation in progress");<a name="line.894"></a>
+<span class="sourceLineNo">895</span>    }<a name="line.895"></a>
+<span class="sourceLineNo">896</span><a name="line.896"></a>
+<span class="sourceLineNo">897</span>    // sanity check on the request<a 
name="line.897"></a>
+<span class="sourceLineNo">898</span>    if (!Bytes.equals(hriA.getEndKey(), 
hriB.getStartKey())) {<a name="line.898"></a>
+<span class="sourceLineNo">899</span>      throw new 
UnsupportedOperationException(<a name="line.899"></a>
+<span class="sourceLineNo">900</span>        "unsupported split request with 
bad keys: parent=" + parent +<a name="line.900"></a>
+<span class="sourceLineNo">901</span>        " hriA=" + hriA + " hriB=" + 
hriB);<a name="line.901"></a>
+<span class="sourceLineNo">902</span>    }<a name="line.902"></a>
+<span class="sourceLineNo">903</span><a name="line.903"></a>
+<span class="sourceLineNo">904</span>    // Submit the Split procedure<a 
name="line.904"></a>
+<span class="sourceLineNo">905</span>    final byte[] splitKey = 
hriB.getStartKey();<a name="line.905"></a>
+<span class="sourceLineNo">906</span>    if (LOG.isDebugEnabled()) {<a 
name="line.906"></a>
+<span class="sourceLineNo">907</span>      LOG.debug("Split request from " + 
serverName +<a name="line.907"></a>
+<span class="sourceLineNo">908</span>          ", parent=" + parent + " 
splitKey=" + Bytes.toStringBinary(splitKey));<a name="line.908"></a>
+<span class="sourceLineNo">909</span>    }<a name="line.909"></a>
+<span class="sourceLineNo">910</span>    
master.getMasterProcedureExecutor().submitProcedure(createSplitProcedure(parent,
 splitKey));<a name="line.910"></a>
+<span class="sourceLineNo">911</span><a name="line.911"></a>
+<span class="sourceLineNo">912</span>    // If the RS is &lt; 2.0 throw an 
exception to abort the operation, we are handling the split<a 
name="line.912"></a>
+<span class="sourceLineNo">913</span>    if 
(master.getServerManager().getVersionNumber(serverName) &lt; 0x0200000) {<a 
name="line.913"></a>
+<span class="sourceLineNo">914</span>      throw new 
UnsupportedOperationException(String.format(<a name="line.914"></a>
+<span class="sourceLineNo">915</span>        "Split handled by the master: 
parent=%s hriA=%s hriB=%s", parent.getShortNameToLog(), hriA, hriB));<a 
name="line.915"></a>
+<span class="sourceLineNo">916</span>    }<a name="line.916"></a>
+<span class="sourceLineNo">917</span>  }<a name="line.917"></a>
+<span class="sourceLineNo">918</span><a name="line.918"></a>
+<span class="sourceLineNo">919</span>  private void 
updateRegionMergeTransition(final ServerName serverName, final TransitionCode 
state,<a name="line.919"></a>
+<span class="sourceLineNo">920</span>      final RegionInfo merged, final 
RegionInfo hriA, final RegionInfo hriB) throws IOException {<a 
name="line.920"></a>
+<span class="sourceLineNo">921</span>    checkMetaLoaded(merged);<a 
name="line.921"></a>
 <span class="sourceLineNo">922</span><a name="line.922"></a>
-<span class="sourceLineNo">923</span>    // If the RS is &lt; 2.0 throw an 
exception to abort the operation, we are handling the merge<a 
name="line.923"></a>
-<span class="sourceLineNo">924</span>    if 
(master.getServerManager().getVersionNumber(serverName) &lt; 0x0200000) {<a 
name="line.924"></a>
-<span class="sourceLineNo">925</span>      throw new 
UnsupportedOperationException(String.format(<a name="line.925"></a>
-<span class="sourceLineNo">926</span>        "Merge not handled yet: 
regionState=%s merged=%s hriA=%s hriB=%s", state, merged, hriA,<a 
name="line.926"></a>
-<span class="sourceLineNo">927</span>          hriB));<a name="line.927"></a>
-<span class="sourceLineNo">928</span>    }<a name="line.928"></a>
-<span class="sourceLineNo">929</span>  }<a name="line.929"></a>
-<span class="sourceLineNo">930</span><a name="line.930"></a>
-<span class="sourceLineNo">931</span>  // 
============================================================================================<a
 name="line.931"></a>
-<span class="sourceLineNo">932</span>  //  RS Status update (report online 
regions) helpers<a name="line.932"></a>
-<span class="sourceLineNo">933</span>  // 
============================================================================================<a
 name="line.933"></a>
-<span class="sourceLineNo">934</span>  /**<a name="line.934"></a>
-<span class="sourceLineNo">935</span>   * the master will call this method 
when the RS send the regionServerReport().<a name="line.935"></a>
-<span class="sourceLineNo">936</span>   * the report will contains the "online 
regions".<a name="line.936"></a>
-<span class="sourceLineNo">937</span>   * this method will check the the 
online regions against the in-memory state of the AM,<a name="line.937"></a>
-<span class="sourceLineNo">938</span>   * if there is a mismatch we will try 
to fence out the RS with the assumption<a name="line.938"></a>
-<span class="sourceLineNo">939</span>   * that something went wrong on the RS 
side.<a name="line.939"></a>
-<span class="sourceLineNo">940</span>   */<a name="line.940"></a>
-<span class="sourceLineNo">941</span>  public void reportOnlineRegions(final 
ServerName serverName, final Set&lt;byte[]&gt; regionNames)<a 
name="line.941"></a>
-<span class="sourceLineNo">942</span>      throws YouAreDeadException {<a 
name="line.942"></a>
-<span class="sourceLineNo">943</span>    if (!isRunning()) return;<a 
name="line.943"></a>
-<span class="sourceLineNo">944</span>    if (LOG.isTraceEnabled()) {<a 
name="line.944"></a>
-<span class="sourceLineNo">945</span>      LOG.trace("ReportOnlineRegions " + 
serverName + " regionCount=" + regionNames.size() +<a name="line.945"></a>
-<span class="sourceLineNo">946</span>        ", metaLoaded=" + isMetaLoaded() 
+ " " +<a name="line.946"></a>
-<span class="sourceLineNo">947</span>          
regionNames.stream().map(element -&gt; Bytes.toStringBinary(element)).<a 
name="line.947"></a>
-<span class="sourceLineNo">948</span>            
collect(Collectors.toList()));<a name="line.948"></a>
-<span class="sourceLineNo">949</span>    }<a name="line.949"></a>
-<span class="sourceLineNo">950</span><a name="line.950"></a>
-<span class="sourceLineNo">951</span>    final ServerStateNode serverNode = 
regionStates.getOrCreateServer(serverName);<a name="line.951"></a>
-<span class="sourceLineNo">952</span><a name="line.952"></a>
-<span class="sourceLineNo">953</span>    synchronized (serverNode) {<a 
name="line.953"></a>
-<span class="sourceLineNo">954</span>      if 
(!serverNode.isInState(ServerState.ONLINE)) {<a name="line.954"></a>
-<span class="sourceLineNo">955</span>        LOG.warn("Got a report from a 
server result in state " + serverNode.getState());<a name="line.955"></a>
-<span class="sourceLineNo">956</span>        return;<a name="line.956"></a>
-<span class="sourceLineNo">957</span>      }<a name="line.957"></a>
-<span class="sourceLineNo">958</span>    }<a name="line.958"></a>
-<span class="sourceLineNo">959</span><a name="line.959"></a>
-<span class="sourceLineNo">960</span>    if (regionNames.isEmpty()) {<a 
name="line.960"></a>
-<span class="sourceLineNo">961</span>      // nothing to do if we don't have 
regions<a name="line.961"></a>
-<span class="sourceLineNo">962</span>      LOG.trace("no online region found 
on " + serverName);<a name="line.962"></a>
-<span class="sourceLineNo">963</span>    } else if (!isMetaLoaded()) {<a 
name="line.963"></a>
-<span class="sourceLineNo">964</span>      // if we are still on startup, 
discard the report unless is from someone holding meta<a name="line.964"></a>
-<span class="sourceLineNo">965</span>      
checkOnlineRegionsReportForMeta(serverNode, regionNames);<a name="line.965"></a>
-<span class="sourceLineNo">966</span>    } else {<a name="line.966"></a>
-<span class="sourceLineNo">967</span>      // The Heartbeat updates us of what 
regions are only. check and verify the state.<a name="line.967"></a>
-<span class="sourceLineNo">968</span>      
checkOnlineRegionsReport(serverNode, regionNames);<a name="line.968"></a>
-<span class="sourceLineNo">969</span>    }<a name="line.969"></a>
-<span class="sourceLineNo">970</span><a name="line.970"></a>
-<span class="sourceLineNo">971</span>    // wake report event<a 
name="line.971"></a>
-<span class="sourceLineNo">972</span>    wakeServerReportEvent(serverNode);<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>  void 
checkOnlineRegionsReportForMeta(final ServerStateNode serverNode,<a 
name="line.975"></a>
-<span class="sourceLineNo">976</span>      final Set&lt;byte[]&gt; 
regionNames) {<a name="line.976"></a>
-<span class="sourceLineNo">977</span>    try {<a name="line.977"></a>
-<span class="sourceLineNo">978</span>      for (byte[] regionName: 
regionNames) {<a name="line.978"></a>
-<span class="sourceLineNo">979</span>        final RegionInfo hri = 
getMetaRegionFromName(regionName);<a name="line.979"></a>
-<span class="sourceLineNo">980</span>        if (hri == null) {<a 
name="line.980"></a>
-<span class="sourceLineNo">981</span>          if (LOG.isTraceEnabled()) {<a 
name="line.981"></a>
-<span class="sourceLineNo">982</span>            LOG.trace("Skip online report 
for region=" + Bytes.toStringBinary(regionName) +<a name="line.982"></a>
-<span class="sourceLineNo">983</span>              " while meta is 
loading");<a name="line.983"></a>
-<span class="sourceLineNo">984</span>          }<a name="line.984"></a>
-<span class="sourceLineNo">985</span>          continue;<a name="line.985"></a>
-<span class="sourceLineNo">986</span>        }<a name="line.986"></a>
-<span class="sourceLineNo">987</span><a name="line.987"></a>
-<span class="sourceLineNo">988</span>        final RegionStateNode regionNode 
= regionStates.getOrCreateRegionStateNode(hri);<a name="line.988"></a>
-<span class="sourceLineNo">989</span>        LOG.info("META REPORTED: " + 
regionNode);<a name="line.989"></a>
-<span class="sourceLineNo">990</span>        if (!reportTransition(regionNode, 
serverNode, TransitionCode.OPENED, 0)) {<a name="line.990"></a>
-<span class="sourceLineNo">991</span>          LOG.warn("META REPORTED but no 
procedure found (complete?); set location=" +<a name="line.991"></a>
-<span class="sourceLineNo">992</span>              
serverNode.getServerName());<a name="line.992"></a>
-<span class="sourceLineNo">993</span>          
regionNode.setRegionLocation(serverNode.getServerName());<a name="line.993"></a>
-<span class="sourceLineNo">994</span>        } else if (LOG.isTraceEnabled()) 
{<a name="line.994"></a>
-<span class="sourceLineNo">995</span>          LOG.trace("META REPORTED: " + 
regionNode);<a name="line.995"></a>
-<span class="sourceLineNo">996</span>        }<a name="line.996"></a>
-<span class="sourceLineNo">997</span>      }<a name="line.997"></a>
-<span class="sourceLineNo">998</span>    } catch (UnexpectedStateException e) 
{<a name="line.998"></a>
-<span class="sourceLineNo">999</span>      final ServerName serverName = 
serverNode.getServerName();<a name="line.999"></a>
-<span class="sourceLineNo">1000</span>      LOG.warn("KILLING " + serverName + 
": " + e.getMessage());<a name="line.1000"></a>
-<span class="sourceLineNo">1001</span>      killRegionServer(serverNode);<a 
name="line.1001"></a>
-<span class="sourceLineNo">1002</span>    }<a name="line.1002"></a>
-<span class="sourceLineNo">1003</span>  }<a name="line.1003"></a>
-<span class="sourceLineNo">1004</span><a name="line.1004"></a>
-<span class="sourceLineNo">1005</span>  void checkOnlineRegionsReport(final 
ServerStateNode serverNode, final Set&lt;byte[]&gt; regionNames)<a 
name="line.1005"></a>
-<span class="sourceLineNo">1006</span>      throws YouAreDeadException {<a 
name="line.1006"></a>
-<span class="sourceLineNo">1007</span>    final ServerName serverName = 
serverNode.getServerName();<a name="line.1007"></a>
-<span class="sourceLineNo">1008</span>    try {<a name="line.1008"></a>
-<span class="sourceLineNo">1009</span>      for (byte[] regionName: 
regionNames) {<a name="line.1009"></a>
-<span class="sourceLineNo">1010</span>        if (!isRunning()) return;<a 
name="line.1010"></a>
-<span class="sourceLineNo">1011</span>        final RegionStateNode regionNode 
= regionStates.getRegionStateNodeFromName(regionName);<a name="line.1011"></a>
-<span class="sourceLineNo">1012</span>        if (regionNode == null) {<a 
name="line.1012"></a>
-<span class="sourceLineNo">1013</span>          throw new 
UnexpectedStateException("Not online: " + Bytes.toStringBinary(regionName));<a 
name="line.1013"></a>
-<span class="sourceLineNo">1014</span>        }<a name="line.1014"></a>
-<span class="sourceLineNo">1015</span>        synchronized (regionNode) {<a 
name="line.1015"></a>
-<span class="sourceLineNo">1016</span>          if 
(regionNode.isInState(State.OPENING, State.OPEN)) {<a name="line.1016"></a>
-<span class="sourceLineNo">1017</span>            if 
(!regionNode.getRegionLocation().equals(serverName)) {<a name="line.1017"></a>
-<span class="sourceLineNo">1018</span>              throw new 
UnexpectedStateException(regionNode.toString() +<a name="line.1018"></a>
-<span class="sourceLineNo">1019</span>                " reported OPEN on 
server=" + serverName +<a name="line.1019"></a>
-<span class="sourceLineNo">1020</span>                " but state has 
otherwise.");<a name="line.1020"></a>
-<span class="sourceLineNo">1021</span>            } else if 
(regionNode.isInState(State.OPENING)) {<a name="line.1021"></a>
-<span class="sourceLineNo">1022</span>              try {<a 
name="line.1022"></a>
-<span class="sourceLineNo">1023</span>                if 
(!reportTransition(regionNode, serverNode, TransitionCode.OPENED, 0)) {<a 
name="line.1023"></a>
-<span class="sourceLineNo">1024</span>                  
LOG.warn(regionNode.toString() + " reported OPEN on server=" + serverName +<a 
name="line.1024"></a>
-<span class="sourceLineNo">1025</span>                    " but state has 
otherwise AND NO procedure is running");<a name="line.1025"></a>
-<span class="sourceLineNo">1026</span>                }<a name="line.1026"></a>
-<span class="sourceLineNo">1027</span>              } catch 
(UnexpectedStateException e) {<a name="line.1027"></a>
-<span class="sourceLineNo">1028</span>                
LOG.warn(regionNode.toString() + " reported unexpteced OPEN: " + 
e.getMessage(), e);<a name="line.1028"></a>
-<span class="sourceLineNo">1029</span>              }<a name="line.1029"></a>
-<span class="sourceLineNo">1030</span>            }<a name="line.1030"></a>
-<span class="sourceLineNo">1031</span>          } else if 
(!regionNode.isInState(State.CLOSING, State.SPLITTING)) {<a 
name="line.1031"></a>
-<span class="sourceLineNo">1032</span>            long diff = 
regionNode.getLastUpdate() - EnvironmentEdgeManager.currentTime();<a 
name="line.1032"></a>
-<span class="sourceLineNo">1033</span>            if (diff &gt; 1000/*One 
Second... make configurable if an issue*/) {<a name="line.1033"></a>
-<span class="sourceLineNo">1034</span>              // So, we can get report 
that a region is CLOSED or SPLIT because a heartbeat<a name="line.1034"></a>
-<span class="sourceLineNo">1035</span>              // came in at about same 
time as a region transition. Make sure there is some<a name="line.1035"></a>
-<span class="sourceLineNo">1036</span>              // elapsed time between 
killing remote server.<a name="line.1036"></a>
-<span class="sourceLineNo">1037</span>              throw new 
UnexpectedStateException(regionNode.toString() +<a name="line.1037"></a>
-<span class="sourceLineNo">1038</span>                " reported an unexpected 
OPEN; time since last update=" + diff);<a name="line.1038"></a>
-<span class="sourceLineNo">1039</span>            }<a name="line.1039"></a>
-<span class="sourceLineNo">1040</span>          }<a name="line.1040"></a>
-<span class="sourceLineNo">1041</span>        }<a name="line.1041"></a>
-<span class="sourceLineNo">1042</span>      }<a name="line.1042"></a>
-<span class="sourceLineNo">1043</span>    } catch (UnexpectedStateException e) 
{<a name="line.1043"></a>
-<span class="sourceLineNo">1044</span>      LOG.warn("Killing " + serverName + 
": " + e.getMessage());<a name="line.1044"></a>
-<span class="sourceLineNo">1045</span>      killRegionServer(serverNode);<a 
name="line.1045"></a>
-<span class="sourceLineNo">1046</span>      throw (YouAreDeadException)new 
YouAreDeadException(e.getMessage()).initCause(e);<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><a name="line.1049"></a>
-<span class="sourceLineNo">1050</span>  protected boolean 
waitServerReportEvent(ServerName serverName, Procedure&lt;?&gt; proc) {<a 
name="line.1050"></a>
-<span class="sourceLineNo">1051</span>    final ServerStateNode serverNode = 
regionStates.getOrCreateServer(serverName);<a name="line.1051"></a>
-<span class="sourceLineNo">1052</span>    if (serverNode == null) {<a 
name="line.1052"></a>
-<span class="sourceLineNo">1053</span>      LOG.warn("serverName=null; {}", 
proc);<a name="line.1053"></a>
-<span class="sourceLineNo">1054</span>    }<a name="line.1054"></a>
-<span class="sourceLineNo">1055</span>    return 
serverNode.getReportEvent().suspendIfNotReady(proc);<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>  protected void 
wakeServerReportEvent(final ServerStateNode serverNode) {<a 
name="line.1058"></a>
-<span class="sourceLineNo">1059</span>    
serverNode.getReportEvent().wake(getProcedureScheduler());<a 
name="line.1059"></a>
+<span class="sourceLineNo">923</span>    if (state != 
TransitionCode.READY_TO_MERGE) {<a name="line.923"></a>
+<span class="sourceLineNo">924</span>      throw new 
UnexpectedStateException("Unsupported merge regionState=" + state +<a 
name="line.924"></a>
+<span class="sourceLineNo">925</span>        " for regionA=" + hriA + " 
regionB=" + hriB + " merged=" + merged +<a name="line.925"></a>
+<span class="sourceLineNo">926</span>        " maybe an old RS (&lt; 2.0) had 
the operation in progress");<a name="line.926"></a>
+<span class="sourceLineNo">927</span>    }<a name="line.927"></a>
+<span class="sourceLineNo">928</span><a name="line.928"></a>
+<span class="sourceLineNo">929</span>    // Submit the Merge procedure<a 
name="line.929"></a>
+<span class="sourceLineNo">930</span>    if (LOG.isDebugEnabled()) {<a 
name="line.930"></a>
+<span class="sourceLineNo">931</span>      LOG.debug("Handling merge request 
from RS=" + merged + ", merged=" + merged);<a name="line.931"></a>
+<span class="sourceLineNo">932</span>    }<a name="line.932"></a>
+<span class="sourceLineNo">933</span>    
master.getMasterProcedureExecutor().submitProcedure(createMergeProcedure(hriA, 
hriB));<a name="line.933"></a>
+<span class="sourceLineNo">934</span><a name="line.934"></a>
+<span class="sourceLineNo">935</span>    // If the RS is &lt; 2.0 throw an 
exception to abort the operation, we are handling the merge<a 
name="line.935"></a>
+<span class="sourceLineNo">936</span>    if 
(master.getServerManager().getVersionNumber(serverName) &lt; 0x0200000) {<a 
name="line.936"></a>
+<span class="sourceLineNo">937</span>      throw new 
UnsupportedOperationException(String.format(<a name="line.937"></a>
+<span class="sourceLineNo">938</span>        "Merge not handled yet: 
regionState=%s merged=%s hriA=%s hriB=%s", state, merged, hriA,<a 
name="line.938"></a>
+<span class="sourceLineNo">939</span>          hriB));<a name="line.939"></a>
+<span class="sourceLineNo">940</span>    }<a name="line.940"></a>
+<span class="sourceLineNo">941</span>  }<a name="line.941"></a>
+<span class="sourceLineNo">942</span><a name="line.942"></a>
+<span class="sourceLineNo">943</span>  // 
============================================================================================<a
 name="line.943"></a>
+<span class="sourceLineNo">944</span>  //  RS Status update (report online 
regions) helpers<a name="line.944"></a>
+<span class="sourceLineNo">945</span>  // 
============================================================================================<a
 name="line.945"></a>
+<span class="sourceLineNo">946</span>  /**<a name="line.946"></a>
+<span class="sourceLineNo">947</span>   * the master will call this method 
when the RS send the regionServerReport().<a name="line.947"></a>
+<span class="sourceLineNo">948</span>   * the report will contains the "online 
regions".<a name="line.948"></a>
+<span class="sourceLineNo">949</span>   * this method will check the the 
online regions against the in-memory state of the AM,<a name="line.949"></a>
+<span class="sourceLineNo">950</span>   * if there is a mismatch we will try 
to fence out the RS with the assumption<a name="line.950"></a>
+<span class="sourceLineNo">951</span>   * that something went wrong on the RS 
side.<a name="line.951"></a>
+<span class="sourceLineNo">952</span>   */<a name="line.952"></a>
+<span class="sourceLineNo">953</span>  public void reportOnlineRegions(final 
ServerName serverName, final Set&lt;byte[]&gt; regionNames)<a 
name="line.953"></a>
+<span class="sourceLineNo">954</span>      throws YouAreDeadException {<a 
name="line.954"></a>
+<span class="sourceLineNo">955</span>    if (!isRunning()) return;<a 
name="line.955"></a>
+<span class="sourceLineNo">956</span>    if (LOG.isTraceEnabled()) {<a 
name="line.956"></a>
+<span class="sourceLineNo">957</span>      LOG.trace("ReportOnlineRegions " + 
serverName + " regionCount=" + regionNames.size() +<a name="line.957"></a>
+<span class="sourceLineNo">958</span>        ", metaLoaded=" + isMetaLoaded() 
+ " " +<a name="line.958"></a>
+<span class="sourceLineNo">959</span>          
regionNames.stream().map(element -&gt; Bytes.toStringBinary(element)).<a 
name="line.959"></a>
+<span class="sourceLineNo">960</span>            
collect(Collectors.toList()));<a name="line.960"></a>
+<span class="sourceLineNo">961</span>    }<a name="line.961"></a>
+<span class="sourceLineNo">962</span><a name="line.962"></a>
+<span class="sourceLineNo">963</span>    final ServerStateNode serverNode = 
regionStates.getOrCreateServer(serverName);<a name="line.963"></a>
+<span class="sourceLineNo">964</span><a name="line.964"></a>
+<span class="sourceLineNo">965</span>    synchronized (serverNode) {<a 
name="line.965"></a>
+<span class="sourceLineNo">966</span>      if 
(!serverNode.isInState(ServerState.ONLINE)) {<a name="line.966"></a>
+<span class="sourceLineNo">967</span>        LOG.warn("Got a report from a 
server result in state " + serverNode.getState());<a name="line.967"></a>
+<span class="sourceLineNo">968</span>        return;<a name="line.968"></a>
+<span class="sourceLineNo">969</span>      }<a name="line.969"></a>
+<span class="sourceLineNo">970</span>    }<a name="line.970"></a>
+<span class="sourceLineNo">971</span><a name="line.971"></a>
+<span class="sourceLineNo">972</span>    if (regionNames.isEmpty()) {<a 
name="line.972"></a>
+<span class="sourceLineNo">973</span>      // nothing to do if we don't have 
regions<a name="line.973"></a>
+<span class="sourceLineNo">974</span>      LOG.trace("no online region found 
on " + serverName);<a name="line.974"></a>
+<span class="sourceLineNo">975</span>    } else if (!isMetaLoaded()) {<a 
name="line.975"></a>
+<span class="sourceLineNo">976</span>      // if we are still on startup, 
discard the report unless is from someone holding meta<a name="line.976"></a>
+<span class="sourceLineNo">977</span>      
checkOnlineRegionsReportForMeta(serverNode, regionNames);<a name="line.977"></a>
+<span class="sourceLineNo">978</span>    } else {<a name="line.978"></a>
+<span class="sourceLineNo">979</span>      // The Heartbeat updates us of what 
regions are only. check and verify the state.<a name="line.979"></a>
+<span class="sourceLineNo">980</span>      
checkOnlineRegionsReport(serverNode, regionNames);<a name="line.980"></a>
+<span class="sourceLineNo">981</span>    }<a name="line.981"></a>
+<span class="sourceLineNo">982</span><a name="line.982"></a>
+<span class="sourceLineNo">983</span>    // wake report event<a 
name="line.983"></a>
+<span class="sourceLineNo">984</span>    wakeServerReportEvent(serverNode);<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>  void 
checkOnlineRegionsReportForMeta(final ServerStateNode serverNode,<a 
name="line.987"></a>
+<span class="sourceLineNo">988</span>      final Set&lt;byte[]&gt; 
regionNames) {<a name="line.988"></a>
+<span class="sourceLineNo">989</span>    try {<a name="line.989"></a>
+<span class="sourceLineNo">990</span>      for (byte[] regionName: 
regionNames) {<a name="line.990"></a>
+<span class="sourceLineNo">991</span>        final RegionInfo hri = 
getMetaRegionFromName(regionName);<a name="line.991"></a>
+<span class="sourceLineNo">992</span>        if (hri == null) {<a 
name="line.992"></a>
+<span class="sourceLineNo">993</span>          if (LOG.isTraceEnabled()) {<a 
name="line.993"></a>
+<span class="sourceLineNo">994</span>            LOG.trace("Skip online report 
for region=" + Bytes.toStringBinary(regionName) +<a name="line.994"></a>
+<span class="sourceLineNo">995</span>              " while meta is 
loading");<a name="line.995"></a>
+<span class="sourceLineNo">996</span>          }<a name="line.996"></a>
+<span class="sourceLineNo">997</span>          continue;<a name="line.997"></a>
+<span class="sourceLineNo">998</span>        }<a name="line.998"></a>
+<span class="sourceLineNo">999</span><a name="line.999"></a>
+<span class="sourceLineNo">1000</span>        final RegionStateNode regionNode 
= regionStates.getOrCreateRegionStateNode(hri);<a name="line.1000"></a>
+<span class="sourceLineNo">1001</span>        LOG.info("META REPORTED: " + 
regionNode);<a name="line.1001"></a>
+<span class="sourceLineNo">1002</span>        if 
(!reportTransition(regionNode, serverNode, TransitionCode.OPENED, 0)) {<a 
name="line.1002"></a>
+<span class="sourceLineNo">1003</span>          LOG.warn("META REPORTED but no 
procedure found (complete?); set location=" +<a name="line.1003"></a>
+<span class="sourceLineNo">1004</span>              
serverNode.getServerName());<a name="line.1004"></a>
+<span class="sourceLineNo">1005</span>          
regionNode.setRegionLocation(serverNode.getServerName());<a 
name="line.1005"></a>
+<span class="sourceLineNo">1006</span>        } else if (LOG.isTraceEnabled()) 
{<a name="line.1006"></a>
+<span class="sourceLineNo">1007</span>          LOG.trace("META REPORTED: " + 
regionNode);<a name="line.1007"></a>
+<span class="sourceLineNo">1008</span>        }<a name="line.1008"></a>
+<span class="sourceLineNo">1009</span>      }<a name="line.1009"></a>
+<span class="sourceLineNo">1010</span>    } catch (UnexpectedStateException e) 
{<a name="line.1010"></a>
+<span class="sourceLineNo">1011</span>      final ServerName serverName = 
serverNode.getServerName();<a name="line.1011"></a>
+<span class="sourceLineNo">1012</span>      LOG.warn("KILLING " + serverName + 
": " + e.getMessage());<a name="line.1012"></a>
+<span class="sourceLineNo">1013</span>      killRegionServer(serverNode);<a 
name="line.1013"></a>
+<span class="sourceLineNo">1014</span>    }<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>  void checkOnlineRegionsReport(final 
ServerStateNode serverNode, final Set&lt;byte[]&gt; regionNames)<a 
name="line.1017"></a>
+<span class="sourceLineNo">1018</span>      throws YouAreDeadException {<a 
name="line.1018"></a>
+<span class="sourceLineNo">1019</span>    final ServerName serverName = 
serverNode.getServerName();<a name="line.1019"></a>
+<span class="sourceLineNo">1020</span>    try {<a name="line.1020"></a>
+<span class="sourceLineNo">1021</span>      for (byte[] regionName: 
regionNames) {<a name="line.1021"></a>
+<span class="sourceLineNo">1022</span>        if (!isRunning()) return;<a 
name="line.1022"></a>
+<span class="sourceLineNo">1023</span>        final RegionStateNode regionNode 
= regionStates.getRegionStateNodeFromName(regionName);<a name="line.1023"></a>
+<span class="sourceLineNo">1024</span>        if (regionNode == null) {<a 
name="line.1024"></a>
+<span class="sourceLineNo">1025</span>          throw new 
UnexpectedStateException("Not online: " + Bytes.toStringBinary(regionName));<a 
name="line.1025"></a>
+<span class="sourceLineNo">1026</span>        }<a name="line.1026"></a>
+<span class="sourceLineNo">1027</span>        synchronized (regionNode) {<a 
name="line.1027"></a>
+<span class="sourceLineNo">1028</span>          if 
(regionNode.isInState(State.OPENING, State.OPEN)) {<a name="line.1028"></a>
+<span class="sourceLineNo">1029</span>            if 
(!regionNode.getRegionLocation().equals(serverName)) {<a name="line.1029"></a>
+<span class="sourceLineNo">1030</span>              throw new 
UnexpectedStateException(regionNode.toString() +<a name="line.1030"></a>
+<span class="sourceLineNo">1031</span>                " reported OPEN on 
server=" + serverName +<a name="line.1031"></a>
+<span class="sourceLineNo">1032</span>                " but state has 
otherwise.");<a name="line.1032"></a>
+<span class="sourceLineNo">1033</span>            } else if 
(regionNode.isInState(State.OPENING)) {<a name="line.1033"></a>
+<span class="sourceLineNo">1034</span>              try {<a 
name="line.1034"></a>
+<span class="sourceLineNo">1035</span>                if 
(!reportTransition(regionNode, serverNode, TransitionCode.OPENED, 0)) {<a 
name="line.1035"></a>
+<span class="sourceLineNo">1036</span>                  
LOG.warn(regionNode.toString() + " reported OPEN on server=" + serverName +<a 
name="line.1036"></a>
+<span class="sourceLineNo">1037</span>                    " but state has 
otherwise AND NO procedure is running");<a name="line.1037"></a>
+<span class="sourceLineNo">1038</span>                }<a name="line.1038"></a>
+<span class="sourceLineNo">1039</span>              } catch 
(UnexpectedStateException e) {<a name="line.1039"></a>
+<span class="sourceLineNo">1040</span>                
LOG.warn(regionNode.toString() + " reported unexpteced OPEN: " + 
e.getMessage(), e);<a name="line.1040"></a>
+<span class="sourceLineNo">1041</span>              }<a name="line.1041"></a>
+<span class="sourceLineNo">1042</span>            }<a name="line.1042"></a>
+<span class="sourceLineNo">1043</span>          } else if 
(!regionNode.isInState(State.CLOSING, State.SPLITTING)) {<a 
name="line.1043"></a>
+<span class="sourceLineNo">1044</span>            long diff = 
regionNode.getLastUpdate() - EnvironmentEdgeManager.currentTime();<a 
name="line.1044"></a>
+<span class="sourceLineNo">1045</span>            if (diff &gt; 1000/*One 
Second... make configurable if an issue*/) {<a name="line.1045"></a>
+<span class="sourceLineNo">1046</span>              // So, we can get report 
that a region is CLOSED or SPLIT because a heartbeat<a name="line.1046"></a>
+<span class="sourceLineNo">1047</span>              // came in at about same 
time as a region transition. Make sure there is some<a name="line.1047"></a>
+<span class="sourceLineNo">1048</span>              // elapsed time between 
killing remote server.<a name="line.1048"></a>
+<span class="sourceLineNo">1049</span>              throw new 
UnexpectedStateException(regionNode.toString() +<a name="line.1049"></a>
+<span class="sourceLineNo">1050</span>                " reported an unexpected 
OPEN; time since last update=" + diff);<a name="line.1050"></a>
+<span class="sourceLineNo">1051</span>            }<a name="line.1051"></a>
+<span class="sourceLineNo">1052</span>          }<a name="line.1052"></a>
+<span class="sourceLineNo">1053</span>        }<a name="line.1053"></a>
+<span class="sourceLineNo">1054</span>      }<a name="line.1054"></a>
+<span class="sourceLineNo">1055</span>    } catch (UnexpectedStateException e) 
{<a name="line.1055"></a>
+<span class="sourceLineNo">1056</span>      LOG.warn("Killing " + serverName + 
": " + e.getMessage());<a name="line.1056"></a>
+<span class="sourceLineNo">1057</span>      killRegionServer(serverNode);<a 
name="line.1057"></a>
+<span class="sourceLineNo">1058</span>      throw (YouAreDeadException)new 
YouAreDeadException(e.getMessage()).initCause(e);<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><a name="line.1061"></a>
-<span class="sourceLineNo">1062</span>  // 
============================================================================================<a
 name="line.1062"></a>
-<span class="sourceLineNo">1063</span>  //  RIT chore<a name="line.1063"></a>
-<span class="sourceLineNo">1064</span>  // 
============================================================================================<a
 name="line.1064"></a>
-<span class="sourceLineNo">1065</span>  private static class 
RegionInTransitionChore extends 
ProcedureInMemoryChore&lt;MasterProcedureEnv&gt; {<a name="line.1065"></a>
-<span class="sourceLineNo">1066</span>    public RegionInTransitionChore(final 
int timeoutMsec) {<a name="line.1066"></a>
-<span class="sourceLineNo">1067</span>      super(timeoutMsec);<a 
name="line.1067"></a>
-<span class="sourceLineNo">1068</span>    }<a name="line.1068"></a>
+<span class="sourceLineNo">1062</span>  protected boolean 
waitServerReportEvent(ServerName serverName, Procedure&lt;?&gt; proc) {<a 
name="line.1062"></a>
+<span class="sourceLineNo">1063</span>    final ServerStateNode serverNode = 
regionStates.getOrCreateServer(serverName);<a name="line.1063"></a>
+<span class="sourceLineNo">1064</span>    if (serverNode == null) {<a 
name="line.1064"></a>
+<span class="sourceLineNo">1065</span>      LOG.warn("serverName=null; {}", 
proc);<a name="line.1065"></a>
+<span class="sourceLineNo">1066</span>    }<a name="line.1066"></a>
+<span class="sourceLineNo">1067</span>    return 
serverNode.getReportEvent().suspendIfNotReady(proc);<a name="line.1067"></a>
+<span class="sourceLineNo">1068</span>  }<a name="line.1068"></a>
 <span class="sourceLineNo">1069</span><a name="line.1069"></a>
-<span class="sourceLineNo">1070</span>    @Override<a name="line.1070"></a>
-<span class="sourceLineNo">1071</span>    protected void periodicExecute(final 
MasterProcedureEnv env) {<a name="line.1071"></a>
-<span class="sourceLineNo">1072</span>      final AssignmentManager am = 
env.getAssignmentManager();<a name="line.1072"></a>
+<span class="sourceLineNo">1070</span>  protected void 
wakeServerReportEvent(final ServerStateNode serverNode) {<a 
name="line.1070"></a>
+<span class="sourceLineNo">1071</span>    
serverNode.getReportEvent().wake(getProcedureScheduler());<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>      final RegionInTransitionStat 
ritStat = am.computeRegionInTransitionStat();<a name="line.1074"></a>
-<span class="sourceLineNo">1075</span>      if 
(ritStat.hasRegionsOverThreshold()) {<a name="line.1075"></a>
-<span class="sourceLineNo">1076</span>        for (RegionState hri: 
ritStat.getRegionOverThreshold()) {<a name="line.1076"></a>
-<span class="sourceLineNo">1077</span>          
am.handleRegionOverStuckWarningThreshold(hri.getRegion());<a 
name="line.1077"></a>
-<span class="sourceLineNo">1078</span>        }<a name="line.1078"></a>
-<span class="sourceLineNo">1079</span>      }<a name="line.1079"></a>
-<span class="sourceLineNo">1080</span><a name="line.1080"></a>
-<span class="sourceLineNo">1081</span>      // update metrics<a 
name="line.1081"></a>
-<span class="sourceLineNo">1082</span>      
am.updateRegionsInTransitionMetrics(ritStat);<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">1074</span>  // 
============================================================================================<a
 name="line.1074"></a>
+<span class="sourceLineNo">1075</span>  //  RIT chore<a name="line.1075"></a>
+<span class="sourceLineNo">1076</span>  // 
============================================================================================<a
 name="line.1076"></a>
+<span class="sourceLineNo">1077</span>  private static class 
RegionInTransitionChore extends 
ProcedureInMemoryChore&lt;MasterProcedureEnv&gt; {<a name="line.1077"></a>
+<span class="sourceLineNo">1078</span>    public RegionInTransitionChore(final 
int timeoutMsec) {<a name="line.1078"></a>
+<span class="sourceLineNo">1079</span>      super(timeoutMsec);<a 
name="line.1079"></a>
+<span class="sourceLineNo">1080</span>    }<a name="line.1080"></a>
+<span class="sourceLineNo">1081</span><a name="line.1081"></a>
+<span class="sourceLineNo">1082</span>    @Override<a name="line.1082"></a>
+<span class="sourceLineNo">1083</span>    protected void periodicExecute(final 
MasterProcedureEnv env) {<a name="line.1083"></a>
+<span class="sourceLineNo">1084</span>      final AssignmentManager am = 
env.getAssignmentManager();<a name="line.1084"></a>
 <span class="sourceLineNo">1085</span><a name="line.1085"></a>
-<span class="sourceLineNo">1086</span>  public RegionInTransitionStat 
computeRegionInTransitionStat() {<a name="line.1086"></a>
-<span class="sourceLineNo">1087</span>    final RegionInTransitionStat rit = 
new RegionInTransitionStat(getConfiguration());<a name="line.1087"></a>
-<span class="sourceLineNo">1088</span>    rit.update(this);<a 
name="line.1088"></a>
-<span class="sourceLineNo">1089</span>    return rit;<a name="line.1089"></a>
-<span class="sourceLineNo">1090</span>  }<a name="line.1090"></a>
-<span class="sourceLineNo">1091</span><a name="line.1091"></a>
-<span class="sourceLineNo">1092</span>  public static class 
RegionInTransitionStat {<a name="line.1092"></a>
-<span class="sourceLineNo">1093</span>    private final int ritThreshold;<a 
name="line.1093"></a>
-<span class="sourceLineNo">1094</span><a name="line.1094"></a>
-<span class="sourceLineNo">1095</span>    private HashMap&lt;String, 
RegionState&gt; ritsOverThreshold = null;<a name="line.1095"></a>
-<span class="sourceLineNo">1096</span>    private long statTimestamp;<a 
name="line.1096"></a>
-<span class="sourceLineNo">1097</span>    private long oldestRITTime = 0;<a 
name="line.1097"></a>
-<span class="sourceLineNo">1098</span>    private int totalRITsTwiceThreshold 
= 0;<a name="line.1098"></a>
-<span class="sourceLineNo">1099</span>    private int totalRITs = 0;<a 
name="line.1099"></a>
-<span class="sourceLineNo">1100</span><a name="line.1100"></a>
-<span class="sourceLineNo">1101</span>    @VisibleForTesting<a 
name="line.1101"></a>
-<span class="sourceLineNo">1102</span>    public RegionInTransitionStat(final 
Configuration conf) {<a name="line.1102"></a>
-<span class="sourceLineNo">1103</span>      this.ritThreshold =<a 
name="line.1103"></a>
-<span class="sourceLineNo">1104</span>        
conf.getInt(METRICS_RIT_STUCK_WARNING_THRESHOLD, 
DEFAULT_RIT_STUCK_WARNING_THRESHOLD);<a name="line.1104"></a>
-<span class="sourceLineNo">1105</span>    }<a name="line.1105"></a>
+<span class="sourceLineNo">1086</span>      final RegionInTransitionStat 
ritStat = am.computeRegionInTransitionStat();<a name="line.1086"></a>
+<span class="sourceLineNo">1087</span>      if 
(ritStat.hasRegionsOverThreshold()) {<a name="line.1087"></a>
+<span class="sourceLineNo">1088</span>        for (RegionState hri: 
ritStat.getRegionOverThreshold()) {<a name="line.1088"></a>
+<span class="sourceLineNo">1089</span>          
am.handleRegionOverStuckWarningThreshold(hri.getRegion());<a 
name="line.1089"></a>
+<span class="sourceLineNo">1090</span>        }<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>      // update metrics<a 
name="line.1093"></a>
+<span class="sourceLineNo">1094</span>      
am.updateRegionsInTransitionMetrics(ritStat);<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><a name="line.1097"></a>
+<span class="sourceLineNo">1098</span>  public RegionInTransitionStat 
computeRegionInTransitionStat() {<a name="line.1098"></a>
+<span class="sourceLineNo">1099</span>    final RegionInTransitionStat rit = 
new RegionInTransitionStat(getConfiguration());<a name="line.1099"></a>
+<span class="sourceLineNo">1100</span>    rit.update(this);<a 
name="line.1100"></a>
+<span class="sourceLineNo">1101</span>    return rit;<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>  public static class 
RegionInTransitionStat {<a name="line.1104"></a>
+<span class="sourceLineNo">1105</span>    private final int ritThreshold;<a 
name="line.1105"></a>
 <span class="sourceLineNo">1106</span><a name="line.1106"></a>
-<span class="sourceLineNo">1107</span>    public int getRITThreshold() {<a 
name="line.1107"></a>
-<span class="sourceLineNo">1108</span>      return ritThreshold;<a 
name="line.1108"></a>
-<span class="sourceLineNo">1109</span>    }<a name="line.1109"></a>
-<span class="sourceLineNo">1110</span><a name="line.1110"></a>
-<span class="sourceLineNo">1111</span>    public long getTimestamp() {<a 
name="line.1111"></a>
-<span class="sourceLineNo">1112</span>      return statTimestamp;<a 
name="line.1112"></a>
-<span class="sourceLineNo">1113</span>    }<a name="line.1113"></a>
-<span class="sourceLineNo">1114</span><a name="line.1114"></a>
-<span class="sourceLineNo">1115</span>    public int getTotalRITs() {<a 
name="line.1115"></a>
-<span class="sourceLineNo">1116</span>      return totalRITs;<a 
name="line.1116"></a>
+<span class="sourceLineNo">1107</span>    private HashMap&lt;String, 
RegionState&gt; ritsOverThreshold = null;<a name="line.1107"></a>
+<span class="sourceLineNo">1108</span>    private long statTimestamp;<a 
name="line.1108"></a>
+<span class="sourceLineNo">1109</span>    private long oldestRITTime = 0;<a 
name="line.1109"></a>
+<span class="sourceLineNo">1110</span>    private int totalRITsTwiceThreshold 
= 0;<a name="line.1110"></a>
+<span class="sourceLineNo">1111</span>    private int totalRITs = 0;<a 
name="line.1111"></a>
+<span class="sourceLineNo">1112</span><a name="line.1112"></a>
+<span class="sourceLineNo">1113</span>    @VisibleForTesting<a 
name="line.1113"></a>
+<span class="sourceLineNo">1114</span>    public RegionInTransitionStat(final 
Configuration conf) {<a name="line.1114"></a>
+<span class="sourceLineNo">1115</span>      this.ritThreshold =<a 
name="line.1115"></a>
+<span class="sourceLineNo">1116</span>        
conf.getInt(METRICS_RIT_STUCK_WARNING_THRESHOLD, 
DEFAULT_RIT_STUCK_WARNING_THRESHOLD);<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>    public long getOldestRITTime() {<a 
name="line.1119"></a>
-<span class="sourceLineNo">1120</span>      return oldestRITTime;<a 
name="line.1120"></a>
+<span class="sourceLineNo">1119</span>    public int getRITThreshold() {<a 
name="line.1119"></a>
+<span class="sourceLineNo">1120</span>      return ritThreshold;<a 
name="line.1120"></a>
 <span class="sourceLineNo">1121</span>    }<a name="line.1121"></a>
 <span class="sourceLineNo">1122</span><a name="line.1122"></a>
-<span class="sourceLineNo">1123</span>    public int 
getTotalRITsOverThreshold() {<a name="line.1123"></a>
-<span class="sourceLineNo">1124</span>      Map&lt;String, RegionState&gt; m = 
this.ritsOverThreshold;<a name="line.1124"></a>
-<span class="sourceLineNo">1125</span>      return m != null ? m.size() : 0;<a 
name="line.1125"></a>
-<span class="sourceLineNo">1126</span>    }<a name="line.1126"></a>
-<span class="sourceLineNo">1127</span><a name="line.1127"></a>
-<span class="sourceLineNo">1128</span>    public boolean 
hasRegionsTwiceOverThreshold() {<a name="line.1128"></a>
-<span class="sourceLineNo">1129</span>      return totalRITsTwiceThreshold 
&gt; 0;<a name="line.1129"></a>
-<span class="sourceLineNo">1130</span>    }<a name="line.1130"></a>
-<span class="sourceLineNo">1131</span><a name="line.1131"></a>
-<span class="sourceLineNo">1132</span>    public boolean 
hasRegionsOverThreshold() {<a name="line.1132"></a>
-<span class="sourceLineNo">1133</span>      Map&lt;String, RegionState&gt; m = 
this.ritsOverThreshold;<a name="line.1133"></a>
-<span class="sourceLineNo">1134</span>      return m != null &amp;&amp; 
!m.isEmpty();<a name="line.1134"></a>
-<span class="sourceLineNo">1135</span>    }<a name="line.1135"></a>
-<span class="sourceLineNo">1136</span><a name="line.1136"></a>
-<span class="sourceLineNo">1137</span>    public Collection&lt;RegionState&gt; 
getRegionOverThreshold() {<a name="line.1137"></a>
-<span class="sourceLineNo">1138</span>      Map&lt;String, RegionState&gt; m = 
this.ritsOverThreshold;<a name="line.1138"></a>
-<span class="sourceLineNo">1139</span>      return m != null? m.values(): 
Collections.emptySet();<a name="line.1139"></a>
-<span class="sourceLineNo">1140</span>    }<a name="line.1140"></a>
-<span class="sourceLineNo">1141</span><a name="line.1141"></a>
-<span class="sourceLineNo">1142</span>    public boolean 
isRegionOverThreshold(final RegionInfo regionInfo) {<a name="line.1142"></a>
-<span class="sourceLineNo">1143</span>      Map&lt;String, RegionState&gt; m = 
this.ritsOverThreshold;<a name="line.1143"></a>
-<span class="sourceLineNo">1144</span>      return m != null &amp;&amp; 
m.containsKey(regionInfo.getEncodedName());<a name="line.1144"></a>
-<span class="sourceLineNo">1145</span>    }<a name="line.1145"></a>
-<span class="sourceLineNo">1146</span><a name="line.1146"></a>
-<span class="sourceLineNo">1147</span>    public boolean 
isRegionTwiceOverThreshold(final RegionInfo regionInfo) {<a 
name="line.1147"></a>
-<span class="sourceLineNo">1148</span>      Map&lt;String, RegionState&gt; m = 
this.ritsOverThreshold;<a name="line.1148"></a>
-<span class="sourceLineNo">1149</span>      if (m == null) return false;<a 
name="line.1149"></a>
-<span class="sourceLineNo">1150</span>      final RegionState state = 
m.get(regionInfo.getEncodedName());<a name="line.1150"></a>
-<span class="sourceLineNo">1151</span>      if (state == null) return false;<a 
name="line.1151"></a>
-<span class="sourceLineNo">1152</span>      return (statTimestamp - 
state.getStamp()) &gt; (ritThreshold * 2);<a name="line.1152"></a>
-<span class="sourceLineNo">1153</span>    }<a name="line.1153"></a>
-<span class="sourceLineNo">1154</span><a name="line.1154"></a>
-<span class="sourceLineNo">1155</span>    protected void update(final 
AssignmentManager am) {<a name="line.1155"></a>
-<span class="sourceLineNo">1156</span>      final RegionStates regionStates = 
am.getRegionStates();<a name="line.1156"></a>
-<span class="sourceLineNo">1157</span>      this.statTimestamp = 
EnvironmentEdgeManager.currentTime();<a name="line.1157"></a>
-<span class="sourceLineNo">1158</span>      
update(regionStates.getRegionsStateInTransition(), statTimestamp);<a 
name="line.1158"></a>
-<span class="sourceLineNo">1159</span>      
update(regionStates.getRegionFailedOpen(), statTimestamp);<a 
name="line.1159"></a>
-<span class="sourceLineNo">1160</span>    }<a name="line.1160"></a>
-<span class="sourceLineNo">1161</span><a name="line.1161"></a>
-<span class="sourceLineNo">1162</span>    private void update(final 
Collection&lt;RegionState&gt; regions, final long currentTime) {<a 
name="line.1162"></a>
-<span class="sourceLineNo">1163</span>      for (RegionState state: regions) 
{<a name="line.1163"></a>
-<span class="sourceLineNo">1164</span>        totalRITs++;<a 
name="line.1164"></a>
-<span class="sourceLineNo">1165</span>        final long ritTime = currentTime 
- state.getStamp();<a name="line.1165"></a>
-<span class="sourceLineNo">1166</span>        if (ritTime &gt; ritThreshold) 
{<a name="line.1166"></a>
-<span class="sourceLineNo">1167</span>          if (ritsOverThreshold == null) 
{<a name="line.1167"></a>
-<span class="sourceLineNo">1168</span>            ritsOverThreshold = new 
HashMap&lt;String, RegionState&gt;();<a name="line.1168"></a>
-<span class="sourceLineNo">1169</span>          }<a name="line.1169"></a>
-<span class="sourceLineNo">1170</span>          
ritsOverThreshold.put(state.getRegion().getEncodedName(), state);<a 
name="line.1170"></a>
-<span class="sourceLineNo">1171</span>          totalRITsTwiceThreshold += 
(ritTime &gt; (ritThreshold * 2)) ? 1 : 0;<a name="line.1171"></a>
-<span class="sourceLineNo">1172</span>        }<a name="line.1172"></a>
-<span class="sourceLineNo">1173</span>        if (oldestRITTime &lt; ritTime) 
{<a name="line.1173"></a>
-<span class="sourceLineNo">1174</span>          oldestRITTime = ritTime;<a 
name="line.1174"></a>
-<span class="sourceLineNo">1175</span>        }<a name="line.1175"></a>
-<span class="sourceLineNo">1176</span>      }<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><a name="line.1179"></a>
-<span class="sourceLineNo">1180</span>  private void 
updateRegionsInTransitionMetrics(final RegionInTransitionStat ritStat) {<a 
name="line.1180"></a>
-<span class="sourceLineNo">1181</span>    
metrics.updateRITOldestAge(ritStat.getOldestRITTime());<a name="line.1181"></a>
-<span class="sourceLineNo">1182</span>    
metrics.updateRITCount(ritStat.getTotalRITs());<a name="line.1182"></a>
-<span class="sourceLineNo">1183</span>    
metrics.updateRITCountOverThreshold(ritStat.getTotalRITsOverThreshold());<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>  private void 
handleRegionOverStuckWarningThreshold(final RegionInfo regionInfo) {<a 
name="line.1186"></a>
-<span class="sourceLineNo">1187</span>    final RegionStateNode regionNode = 
regionStates.getRegionStateNode(regionInfo);<a name="line.1187"></a>
-<span class="sourceLineNo">1188</span>    //if (regionNode.isStuck()) {<a 
name="line.1188"></a>
-<span class="sourceLineNo">1189</span>    LOG.warn("STUCK Region-In-Transition 
{}", regionNode);<a name="line.1189"></a>
+<span class="sourceLineNo">1123</span>    public long getTimestamp() {<a 
name="line.1123"></a>
+<span class="sourceLineNo">1124</span>      return statTimestamp;<a 
name="line.1124"></a>
+<span class="sourceLineNo">1125</span>    }<a name="line.1125"></a>
+<span class="sourceLineNo">1126</span><a name="line.1126"></a>
+<span class="sourceLineNo">1127</span>    public int getTotalRITs() {<a 
name="line.1127"></a>
+<span class="sourceLineNo">1128</span>      return totalRITs;<a 
name="line.1128"></a>
+<span class="sourceLineNo">1129</span>    }<a name="line.1129"></a>
+<span class="sourceLineNo">1130</span><a name="line.1130"></a>
+<span class="sourceLineNo">1131</span>    public long getOldestRITTime() {<a 
name="line.1131"></a>
+<span class="sourceLineNo">1132</span>      return oldestRITTime;<a 
name="line.1132"></a>
+<span class="sourceLineNo">1133</span>    }<a name="line.1133"></a>
+<span class="sourceLineNo">1134</span><a name="line.1134"></a>
+<span class="sourceLineNo">1135</span>    public int 
getTotalRITsOverThreshold() {<a name="line.1135"></a>
+<span class="sourceLineNo">1136</span>      Map&lt;String, RegionState&gt; m = 
this.ritsOverThreshold;<a name="line.1136"></a>
+<span class="sourceLineNo">1137</span>      return m != null ? m.size() : 0;<a 
name="line.1137"></a>
+<span class="sourceLineNo">1138</span>    }<a name="line.1138"></a>
+<span class="sourceLineNo">1139</span><a name="line.1139"></a>
+<span class="sourceLineNo">1140</span>    public boolean 
hasRegionsTwiceOverThreshold() {<a name="line.1140"></a>
+<span class="sourceLineNo">1141</span>      return totalRITsTwiceThreshold 
&gt; 0;<a name="line.1141"></a>
+<span class="sourceLineNo">1142</span>    }<a name="line.1142"></a>
+<span class="sourceLineNo">1143</span><a name="line.1143"></a>
+<span class="sourceLineNo">1144</span>    public boolean 
hasRegionsOverThreshold() {<a name="line.1144"></a>
+<span class="sourceLineNo">1145</span>      Map&lt;String, RegionState&gt; m = 
this.ritsOverThreshold;<a name="line.1145"></a>
+<span class="sourceLineNo">1146</span>      return m != null &amp;&amp; 
!m.isEmpty();<a name="line.1146"></a>
+<span class="sourceLineNo">1147</span>    }<a name="line.1147"></a>
+<span class="sourceLineNo">1148</span><a name="line.1148"></a>
+<span class="sourceLineNo">1149</span>    public Collection&lt;RegionState&gt; 
getRegionOverThreshold() {<a name="line.1149"></a>
+<span class="sourceLineNo">1150</span>      Map&lt;String, RegionState&gt; m = 
this.ritsOverThreshold;<a name="line.1150"></a>
+<span class="sourceLineNo">1151</span>      return m != null? m.values(): 
Collections.emptySet();<a name="line.1151"></a>
+<span class="sourceLineNo">1152</span>    }<a name="line.1152"></a>
+<span class="sourceLineNo">1153</span><a name="line.1153"></a>
+<span class="sourceLineNo">1154</span>    public boolean 
isRegionOverThreshold(final RegionInfo regionInfo) {<a name="line.1154"></a>
+<span class="sourceLineNo">1155</span>      Map&lt;String, RegionState&gt; m = 
this.ritsOverThreshold;<a name="line.1155"></a>
+<span class="sourceLineNo">1156</span>      return m != null &amp;&amp; 
m.containsKey(regionInfo.getEncodedName());<a name="line.1156"></a>
+<span class="sourceLineNo">1157</span>    }<a name="line.1157"></a>
+<span class="sourceLineNo">1158</span><a name="line.1158"></a>
+<span class="sourceLineNo">1159</span>    public boolean 
isRegionTwiceOverThreshold(final RegionInfo regionInfo) {<a 
name="line.1159"></a>
+<span class="sourceLineNo">1160</span>      Map&lt;String, RegionState&gt; m = 
this.ritsOverThreshold;<a name="line.1160"></a>
+<span class="sourceLineNo">1161</span>      if (m == null) return false;<a 
name="line.1161"></a>
+<span class="sourceLineNo">1162</span>      final RegionState state = 
m.get(regionInfo.getEncodedName());<a name="line.1162"></a>
+<span class="sourceLineNo">1163</span>      if (state == null) return false;<a 
name="line.1163"></a>
+<span class="sourceLineNo">1164</span>      return (statTimestamp - 
state.getStamp()) &gt; (ritThreshold * 2);<a name="line.1164"></a>
+<span class="sourceLineNo">1165</span>    }<a name="line.1165"></a>
+<span class="sourceLineNo">1166</span><a name="line.1166"></a>
+<span class="sourceLineNo">1167</span>    protected void update(final 
AssignmentManager am) {<a name="line.1167"></a>
+<span class="sourceLineNo">1168</span>      final RegionStates regionStates = 
am.getRegionStates();<a name="line.1168"></a>
+<span class="sourceLineNo">1169</span>      this.statTimestamp = 
EnvironmentEdgeManager.currentTime();<a name="line.1169"></a>
+<span class="sourceLineNo">1170</span>      
update(regionStates.getRegionsStateInTransition(), statTimestamp);<a 
name="line.1170"></a>
+<span class="sourceLineNo">1171</span>      
update(regionStates.getRegionFailedOpen(), statTimestamp);<a 
name="line.1171"></a>
+<span class="sourceLineNo">1172</span>    }<a name="line.1172"></a>
+<span class="sourceLineNo">1173</span><a name="line.1173"></a>
+<span class="sourceLineNo">1174</span>    private void update(final 
Collection&lt;RegionState&gt; regions, final long currentTime) {<a 
name="line.1174"></a>
+<span class="sourceLineNo">1175</span>      for (RegionState state: regions) 
{<a name="line.1175"></a>
+<span class="sourceLineNo">1176</span>        totalRITs++;<a 
name="line.1176"></a>
+<span class="sourceLineNo">1177</span>        final long ritTime = currentTime 
- state.getStamp();<a name="line.1177"></a>
+<span class="sourceLineNo">1178</span>        if (ritTime &gt; ritThreshold) 
{<a name="line.1178"></a>
+<span class="sourceLineNo">1179</span>          if (ritsOverThreshold == null) 
{<a name="line.1179"></a>
+<span class="sourceLineNo">1180</span>            ritsOverThreshold = new 
HashMap&lt;String, RegionState&gt;();<a name="line.1180"></a>
+<span class="sourceLineNo">1181</span>          }<a name="line.1181"></a>
+<span class="sourceLineNo">1182</span>          
ritsOverThreshold.put(state.getRegion().getEncodedName(), state);<a 
name="line.1182"></a>
+<span class="sourceLineNo">1183</span>          totalRITsTwiceThreshold += 
(ritTime &gt; (ritThreshold * 2)) ? 1 : 0;<a name="line.1183"></a>
+<span class="sourceLineNo">1184</span>        }<a name="line.1184"></a>
+<span class="sourceLineNo">1185</span>        if (oldestRITTime &lt; ritTime) 
{<a name="line.1185"></a>
+<span class="sourceLineNo">1186</span>          oldestRITTime = ritTime;<a 
name="line.1186"></a>
+<span class="sourceLineNo">1187</span>        }<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>  }<a name="line.1190"></a>
 <span class="sourceLineNo">1191</span><a name="line.1191"></a>
-<span class="sourceLineNo">1192</span>  // 
============================================================================================<a
 name="line.1192"></a>
-<span class="sourceLineNo">1193</span>  //  TODO: Master load/bootstrap<a 
name="line.1193"></a>
-<span class="sourceLineNo">1194</span>  // 
============================================================================================<a
 name="line.1194"></a>
-<span class="sourceLineNo">1195</span>  public void joinCluster() throws 
IOException {<a name="line.1195"></a>
-<span class="sourceLineNo">1196</span>    long startTime = 
System.nanoTime();<a name="line.1196"></a>
-<span class="sourceLineNo">1197</span>    LOG.debug("Joining cluster...");<a 
name="line.1197"></a>
-<span class="sourceLineNo">1198</span><a name="line.1198"></a>
-<span class="sourceLineNo">1199</span>    // Scan hbase:meta to build list of 
existing regions, servers, and assignment<a name="line.1199"></a>
-<span class="sourceLineNo">1200</span>    // hbase:meta is online when we get 
to here and TableStateManager has been started.<a name="line.1200"></a>
-<span class="sourceLineNo">1201</span>    loadMeta();<a name="line.1201"></a>
-<span class="sourceLineNo">1202</span><a name="line.1202"></a>
-<span class="sourceLineNo">1203</span>    while 
(master.getServerManager().countOfRegionServers() &lt; 1) {<a 
name="line.1203"></a>
-<span class="sourceLineNo">1204</span>      LOG.info("Waiting for 
RegionServers to join; current count={}",<a name="line.1204"></a>
-<span class="sourceLineNo">1205</span>        
master.getServerManager().countOfRegionServers());<a name="line.1205"></a>
-<span class="sourceLineNo">1206</span>      Threads.sleep(250);<a 
name="line.1206"></a>
-<span class="sourceLineNo">1207</span>    }<a name="line.1207"></a>
-<span class="sourceLineNo">1208</span>    LOG.info("Number of 
RegionServers={}", master.getServerManager().countOfRegionServers());<a 
name="line.1208"></a>
-<span class="sourceLineNo">1209</span><a name="line.1209"></a>
-<span class="sourceLineNo">1210</span>    processOfflineRegions();<a 
name="line.1210"></a>
-<span class="sourceLineNo">1211</span><a name="line.1211"></a>
-<span class="sourceLineNo">1212</span>    // Start the RIT chore<a 
name="line.1212"></a>
-<span class="sourceLineNo">1213</span>    
master.getMasterProcedureExecutor().addChore(this.ritChore);<a 
name="line.1213"></a>
+<span class="sourceLineNo">1192</span>  private void 
updateRegionsInTransitionMetrics(final RegionInTransitionStat ritStat) {<a 
name="line.1192"></a>
+<span class="sourceLineNo">1193</span>    
metrics.updateRITOldestAge(ritStat.getOldestRITTime());<a name="line.1193"></a>
+<span class="sourceLineNo">1194</span>    
metrics.updateRITCount(ritStat.getTotalRITs());<a name="line.1194"></a>
+<span class="sourceLineNo">1195</span>    
metrics.updateRITCountOverThreshold(ritStat.getTotalRITsOverThreshold());<a 
name="line.1195"></a>
+<span class="sourceLineNo">1196</span>  }<a name="line.1196"></a>
+<span class="sourceLineNo">1197</span><a name="line.1197"></a>
+<span class="sourceLineNo">1198</span>  private void 
handleRegionOverStuckWarningThreshold(final RegionInfo regionInfo) {<a 
name="line.1198"></a>
+<span class="sourceLineNo">1199</span>    final RegionStateNode regionNode = 
regionStates.getRegionStateNode(regionInfo);<a name="line.1199"></a>
+<span class="sourceLineNo">1200</span>    //if (regionNode.isStuck()) {<a 
name="line.1200"></a>
+<span class="sourceLineNo">1201</span>    LOG.warn("STUCK Region-In-Transition 
{}", regionNode);<a name="line.1201"></a>
+<span class="sourceLineNo">1202</span>  }<a name="line.1202"></a>
+<span class="sourceLineNo">1203</span><a name="line.1203"></a>
+<span class="sourceLineNo">1204</span>  // 
============================================================================================<a
 name="line.1204"></a>
+<span class="sourceLineNo">1205</span>  //  TODO: Master load/bootstrap<a 
name="line.1205"></a>
+<span class="sourceLineNo">1206</span>  // 
============================================================================================<a
 name="line.1206"></a>
+<span class="sourceLineNo">1207</span>  public void joinCluster() throws 
IOException {<a name="line.1207"></a>
+<span class="sourceLineNo">1208</span>    long startTime = 
System.nanoTime();<a name="line.1208"></a>
+<span class="sourceLineNo">1209</span>    LOG.debug("Joining cluster...");<a 
name="line.1209"></a>
+<span class="sourceLineNo">1210</span><a name="line.1210"></a>
+<span class="sourceLineNo">1211</span>    // Scan hbase:meta to build list of 
existing regions, servers, and assignment<a name="line.1211"></a>
+<span class="sourceLineNo">1212</span>    // hbase:meta is online when we get 
to here and TableStateManager has been started.<a name="line.1212"></a>
+<span class="sourceLineNo">1213</span>    loadMeta();<a name="line.1213"></a>
 <span class="sourceLineNo">1214</span><a name="line.1214"></a>
-<span class="sourceLineNo">1215</span>    long costMs = 
TimeUnit.NANOSECONDS.toMillis(System.nanoTime() - startTime);<a 
name="line.1215"></a>
-<span class="sourceLineNo">1216</span>    LOG.info("Joined the cluster in {}", 
StringUtils.humanTimeDiff(costMs));<a name="line.1216"></a>
-<span class="sourceLineNo">1217</span>  }<a name="line.1217"></a>
-<span class="sourceLineNo">1218</span><a name="line.1218"></a>
-<span class="sourceLineNo">1219</span>  // Create assign procedure for offline 
regions.<a name="line.1219"></a>
-<span class="sourceLineNo">1220</span>  // Just follow the old 
processofflineServersWithOnlineRegions method. Since now we do not need to<a 
name="line.1220"></a>
-<span class="sourceLineNo">1221</span>  // deal with dead server any more, we 
only deal with the regions in OFFLINE state in this method.<a 
name="line.1221"></a>
-<span class="sourceLineNo">1222</span>  // And this is a bit strange, that for 
new regions, we will add it in CLOSED state instead of<a name="line.1222"></a>
-<span class="sourceLineNo">1223</span>  // OFFLINE state, and usually there 
will be a procedure to track them. The<a name="line.1223"></a>
-<span class="sourceLineNo">1224</span>  // 
processofflineServersWithOnlineRegions is a legacy from long ago, as things are 
going really<a name="line.1224"></a>
-<span class="sourceLineNo">1225</span>  // different now, maybe we do not need 
this method any more. Need to revisit later.<a name="line.1225"></a>
-<span class="sourceLineNo">1226</span>  private void processOfflineRegions() 
{<a name="line.1226"></a>
-<span class="sourceLineNo">1227</span>    List&lt;RegionInfo&gt; 
offlineRegions = regionStates.getRegionStates().stream()<a name="line.1227"></a>
-<span class="sourceLineNo">1228</span>      
.filter(RegionState::isOffline).filter(s -&gt; 
isTableEnabled(s.getRegion().getTable()))<a name="line.1228"></a>
-<span class="sourceLineNo">1229</span>      
.map(RegionState::getRegion).collect(Collectors.toList());<a 
name="line.1229"></a>
-<span class="sourceLineNo">1230</span>    if (!offlineRegions.isEmpty()) {<a 
name="line.1230"></a>
-<span class="sourceLineNo">1231</span>      
master.getMasterProcedureExecutor().submitProcedures(<a name="line.1231"></a>
-<span class="sourceLineNo">1232</span>        
master.getAssignmentManager().createRoundRobinAssignProcedures(offlineRegions));<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><a name="line.1235"></a>
-<span class="sourceLineNo">1236</span>  private void loadMeta() throws 
IOException {<a name="line.1236"></a>
-<span class="sourceLineNo">1237</span>    // TODO: use a thread pool<a 
name="line.1237"></a>
-<span class="sourceLineNo">1238</span>    regionStateStore.visitMeta(new 
RegionStateStore.RegionStateVisitor() {<a name="line.1238"></a>
-<span class="sourceLineNo">1239</span>      @Override<a name="line.1239"></a>
-<span class="sourceLineNo">1240</span>      public void 
visitRegionState(Result result, final RegionInfo regionInfo, final State 
state,<a name="line.1240"></a>
-<span class="sourceLineNo">1241</span>          final ServerName 
regionLocation, final ServerName lastHost, final long openSeqNum) {<a 
name="line.1241"></a>
-<span class="sourceLineNo">1242</span>        if (state == null &amp;&amp; 
regionLocation == null &amp;&amp; lastHost == null &amp;&amp;<a 
name="line.1242"></a>
-<span class="sourceLineNo">1243</span>            openSeqNum == 
SequenceId.NO_SEQUENCE_ID) {<a name="line.1243"></a>
-<span class="sourceLineNo">1244</span>          // This is a row with nothing 
in it.<a name="line.1244"></a>
-<span class="sourceLineNo">1245</span>          LOG.warn("Skipping empty 
row={}", result);<a name="line.1245"></a>
-<span class="sourceLineNo">1246</span>          return;<a name="line.1246"></a>
-<span class="sourceLineNo">1247</span>        }<a name="line.1247"></a>
-<span class="sourceLineNo">1248</span>        State localState = state;<a 
name="line.1248"></a>
-<span class="sourceLineNo">1249</span>        if (localState == null) {<a 
name="line.1249"></a>
-<span class="sourceLineNo">1250</span>          // No region state column data 
in hbase:meta table! Are I doing a rolling upgrade from<a name="line.1250"></a>
-<span class="sourceLineNo">1251</span>          // hbase1 to hbase2? Am I 
restoring a SNAPSHOT or otherwise adding a region to hbase:meta?<a 
name="line.1251"></a>
-<span class="sourceLineNo">1252</span>          // In any of these cases, 
state is empty. For now, presume OFFLINE but there are probably<a 
name="line.1252"></a>
-<span class="sourceLineNo">1253</span>          // cases where we need to 
probe more to be sure this correct; TODO informed by experience.<a 
name="line.1253"></a>
-<span class="sourceLineNo">1254</span>          
LOG.info(regionInfo.getEncodedName() + " regionState=null; presuming " + 
State.OFFLINE);<a name="line.1254"></a>
-<span class="sourceLineNo">1255</span><a name="line.1255"></a>
-<span class="sourceLineNo">1256</span>          localState = State.OFFLINE;<a 
name="line.1256"></a>
-<span class="sourceLineNo">1257</span>        }<a name="line.1257"></a>
-<span class="sourceLineNo">1258</span>        final RegionStateNode regionNode 
= regionStates.getOrCreateRegionStateNode(regionInfo);<a name="line.1258"></a>
-<span class="sourceLineNo">1259</span>        synchronized (regionNode) {<a 
name="line.1259"></a>
-<span class="sourceLineNo">1260</span>          if 
(!regionNode.isInTransition()) {<a name="line.1260"></a>
-<span class="sourceLineNo">1261</span>            
regionNode.setState(localState);<a name="line.1261"></a>
-<span class="sourceLineNo">1262</span>            
regionNode.setLastHost(lastHost);<a name="line.1262"></a>
-<span class="sourceLineNo">1263</span>            
regionNode.setRegionLocation(regionLocation);<a name="line.1263"></a>
-<span class="sourceLineNo">1264</span>            
regionNode.setOpenSeqNum(openSeqNum);<a name="line.1264"></a>
-<span class="sourceLineNo">1265</span><a name="line.1265"></a>
-<span class="sourceLineNo">1266</span>            if (localState == 
State.OPEN) {<a name="line.1266"></a>
-<span class="sourceLineNo">1267</span>              assert regionLocation != 
null : "found null region location for " + regionNode;<a name="line.1267"></a>
-<span class="sourceLineNo">1268</span>              
regionStates.addRegionToServer(regionNode);<a name="line.1268"></a>
-<span class="sourceLineNo">1269</span>            } else if (localState == 
State.OFFLINE || regionInfo.isOffline()) {<a name="line.1269"></a>
-<span class="sourceLineNo">1270</span>              
regionStates.addToOfflineRegions(regionNode);<a name="line.1270"></a>
-<span class="sourceLineNo">1271</span>            } else if (localState == 
State.CLOSED &amp;&amp; getTableStateManager().<a name="line.1271"></a>
-<span class="sourceLineNo">1272</span>                
isTableState(regionNode.getTable(), TableState.State.DISABLED,<a 
name="line.1272"></a>
-<span class="sourceLineNo">1273</span>                
TableState.State.DISABLING)) {<a name="line.1273"></a>
-<span class="sourceLineNo">1274</span>              // The region is CLOSED 
and the table is DISABLED/ DISABLING, there is nothing to<a 
name="line.1274"></a>
-<span class="sourceLineNo">1275</span>              // schedule; the region is 
inert.<a name="line.1275"></a>
-<span class="sourceLineNo">1276</span>            } else {<a 
name="line.1276"></a>
-<span class="sourceLineNo">1277</span>              // These regions should 
have a procedure in replay<a name="line.1277"></a>
-<span class="sourceLineNo">1278</span>              
regionStates.addRegionInTransition(regionNode, null);<a name="line.1278"></a>
-<span class="sourceLineNo">1279</span>            }<a name="line.1279"></a>
-<span class="sourceLineNo">1280</span>          }<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>    // every assignment is blocked until 
meta is loaded.<a name="line.1285"></a>
-<span class="sourceLineNo">1286</span>    wakeMetaLoadedEvent();<a 
name="line.1286"></a>
-<span class="sourceLineNo">1287</span>  }<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>   * Used to check if the meta loading 
is done.<a name="line.1290"></a>
-<span class="sourceLineNo">1291</span>   * &lt;p/&gt;<a name="line.1291"></a>
-<span class="sourceLineNo">1292</span>   * if not we throw PleaseHoldException 
since we are rebuilding the RegionStates<a name="line.1292"></a>
-<span class="sourceLineNo">1293</span>   * @param hri region to check if it is 
already rebuild<a name="line.1293"></a>
-<span class="sourceLineNo">1294</span>   * @throws PleaseHoldException if meta 
has not been loaded yet<a name="line.1294"></a>
-<span class="sourceLineNo">1295</span>   */<a name="line.1295"></a>
-<span class="sourceLineNo">1296</span>  private void 
checkMetaLoaded(RegionInfo hri) throws PleaseHoldException {<a 
name="line.1296"></a>
-<span class="sourceLineNo">1297</span>    if (!isRunning()) {<a 
name="line.1297"></a>
-<span class="sourceLineNo">1298</span>      throw new 
PleaseHoldException("AssignmentManager not running");<a name="line.1298"></a>
-<span class="sourceLineNo">1299</span>    }<a name="line.1299"></a>
-<span class="sourceLineNo">1300</span>    boolean meta = isMetaRegion(hri);<a 
name="line.1300"></a>
-<span class="sourceLineNo">1301</span>    boolean metaLoaded = 
isMetaLoaded();<a name="line.1301"></a>
-<span class="sourceLineNo">1302</span>    if (!meta &amp;&amp; !metaLoaded) 
{<a name="line.1302"></a>
-<span class="sourceLineNo">1303</span>      throw new PleaseHoldException(<a 
name="line.1303"></a>
-<span class="sourceLineNo">1304</span>        "Master not fully online; 
hbase:meta=" + meta + ", metaLoaded=" + metaLoaded);<a name="line.1304"></a>
-<span class="sourceLineNo">1305</span>    }<a name="line.1305"></a>
-<span class="sourceLineNo">1306</span>  }<a name="line.1306"></a>
-<span class="sourceLineNo">1307</span><a name="line.1307"></a>
-<span class="sourceLineNo">1308</span>  // 
============================================================================================<a
 name="line.1308"></a>
-<span class="sourceLineNo">1309</span>  //  TODO: Metrics<a 
name="line.1309"></a>
-<span class="sourceLineNo">1310</span>  // 
============================================================================================<a
 name="line.1310"></a>
-<span class="sourceLineNo">1311</span>  public int getNumRegionsOpened() {<a 
name="line.1311"></a>
-<span class="sourceLineNo">1312</span>    // TODO: Used by 
TestRegionPlacement.java and assume monotonically increasing value<a 
name="line.1312"></a>
-<span class="sourceLineNo">1313</span>    return 0;<a name="line.1313"></a>
-<span class="sourceLineNo">1314</span>  }<a name="line.1314"></a>
-<span class="sourceLineNo">1315</span><a name="line.1315"></a>
-<span class="sourceLineNo">1316</span>  public void submitServerCrash(final 
ServerName serverName, final boolean shouldSplitWal) {<a name="line.1316"></a>
-<span class="sourceLineNo">1317</span>    boolean carryingMeta = 
isCarryingMeta(serverName);<a name="line.1317"></a>
-<span class="sourceLineNo">1318</span>    
ProcedureExecutor&lt;MasterProcedureEnv&gt; procExec = 
this.master.getMasterProcedureExecutor();<a name="line.1318"></a>
-<span class="sourceLineNo">1319</span>    procExec.submitProcedure(new 
ServerCrashProcedure(procExec.getEnvironment(), serverName,<a 
name="line.1319"></a>
-<span class="sourceLineNo">1320</span>      shouldSplitWal, carryingMeta));<a 
name="line.1320"></a>
-<span class="sourceLineNo">1321</span>    LOG.debug("Added=" + serverName +<a 
name="line.1321"></a>
-<span class="sourceLineNo">1322</span>      " to dead servers, submitted 
shutdown handler to be executed meta=" + carryingMeta);<a name="line.1322"></a>
-<span class="sourceLineNo">1323</span>  }<a name="line.1323"></a>
-<span class="sourceLineNo">1324</span><a name="line.1324"></a>
-<span class="sourceLineNo">1325</span>  public void offlineRegion(final 
RegionInfo regionInfo) {<a name="line.1325"></a>
-<span class="sourceLineNo">1326</span>    // TODO used by MasterRpcServices 
ServerCrashProcedure<a name="line.1326"></a>
-<span class="sourceLineNo">1327</span>    final RegionStateNode node = 
regionStates.getRegionStateNode(regionInfo);<a name="line.1327"></a>
-<span class="sourceLineNo">1328</span>    if (node != null) node.offline();<a 
name="line.1328"></a>
-<span class="sourceLineNo">1329</span>  }<a name="line.1329"></a>
-<span class="sourceLineNo">1330</span><a name="line.1330"></a>
-<span class="sourceLineNo">1331</span>  public void onlineRegion(final 
RegionInfo regionInfo, final ServerName serverName) {<a name="line.1331"></a>
-<span class="sourceLineNo">1332</span>    // TODO used by 
TestSplitTransactionOnCluster.java<a name="line.1332"></a>
-<span class="sourceLineNo">1333</span>  }<a name="line.1333"></a>
-<span class="sourceLineNo">1334</span><a name="line.1334"></a>
-<span class="sourceLineNo">1335</span>  public Map&lt;ServerName, 
List&lt;RegionInfo&gt;&gt; getSnapShotOfAssignment(<a name="line.1335"></a>
-<span class="sourceLineNo">1336</span>      final Collection&lt;RegionInfo&gt; 
regions) {<a name="line.1336"></a>
-<span class="sourceLineNo">1337</span>    return 
regionStates.getSnapShotOfAssignment(regions);<a name="line.1337"></a>
-<span class="sourceLineNo">1338</span>  }<a name="line.1338"></a>
-<span class="sourceLineNo">1339</span><a name="line.1339"></a>
-<span

<TRUNCATED>