ramkrish86 commented on a change in pull request #652: HBASE-23035 Retain 
region to the last RegionServer make the failover …
URL: https://github.com/apache/hbase/pull/652#discussion_r328448065
 
 

 ##########
 File path: 
hbase-server/src/main/java/org/apache/hadoop/hbase/master/assignment/TransitRegionStateProcedure.java
 ##########
 @@ -399,6 +399,7 @@ public void reportTransition(MasterProcedureEnv env, 
RegionStateNode regionNode,
   public void serverCrashed(MasterProcedureEnv env, RegionStateNode regionNode,
       ServerName serverName) throws IOException {
     // force to assign to a new candidate server
+    // AssignmentManager#regionClosedAbnormally will set region location to 
null
     // TODO: the forceNewPlan flag not be persistent so if master crash then 
the flag will be lost.
     // But assign to old server is not big deal because it not effect 
correctness.
     // See HBASE-23035 for more details.
 
 Review comment:
   Already some commit has been done for this JIRA?

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to