infraio commented on a change in pull request #221: HBASE-22365 Region may be
opened on two RegionServers
URL: https://github.com/apache/hbase/pull/221#discussion_r282724012
##########
File path:
hbase-server/src/main/java/org/apache/hadoop/hbase/master/assignment/RegionRemoteProcedureBase.java
##########
@@ -175,7 +176,7 @@ void reportTransition(MasterProcedureEnv env,
RegionStateNode regionNode, Server
throw new UnexpectedStateException("Received report from " + serverName
+ ", expected " +
targetServer + ", " + regionNode + ", proc=" + this);
}
- reportTransition(regionNode, transitionCode, seqId);
+ reportTransition(env, regionNode, transitionCode, seqId);
// this state means we have received the report from RS, does not mean the
result is fine, as we
Review comment:
Here we will first change the memory state and then persist to procedure
store. If persist to procedure failed, need to change the memory state back?
----------------------------------------------------------------
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
