bshashikant commented on a change in pull request #1953:
URL: https://github.com/apache/ozone/pull/1953#discussion_r583477482
##########
File path:
hadoop-hdds/server-scm/src/main/java/org/apache/hadoop/hdds/scm/server/StorageContainerManager.java
##########
@@ -1169,11 +1168,12 @@ public ReplicationManager getReplicationManager() {
}
/**
- * Check if the current scm is the leader.
- * @return - if the current scm is the leader.
+ * Check if the current scm is the leader and ready for accepting requests.
+ * @return - if the current scm is the leader and is ready.
*/
public boolean checkLeader() {
- return scmContext.isLeader();
+ return scmContext.isLeader() && getScmHAManager().getRatisServer()
Review comment:
Its for a case, where ratis server is not initialised yet. By default
scmContext.isLeader() will be false.
----------------------------------------------------------------
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
