virajjasani commented on a change in pull request #3826:
URL: https://github.com/apache/hbase/pull/3826#discussion_r745029546



##########
File path: 
hbase-server/src/main/java/org/apache/hadoop/hbase/master/HMaster.java
##########
@@ -938,6 +938,8 @@ private void finishActiveMasterInitialization(MonitoredTask 
status)
     // Set master as 'initialized'.
     setInitialized(true);
 
+    this.assignmentManager.deleteNonZkBasedQualifiersForZkBasedAssignment();

Review comment:
       By this time, rebuildUserRegions() would have already built region 
states in memory (but we have guarding in place). Hence, assignment would not 
interfere with this operation because `info:sn` and `info:state` won't even be 
read for ZK based assignment.




-- 
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.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to