hanishakoneru commented on a change in pull request #1494:
URL: https://github.com/apache/ozone/pull/1494#discussion_r544743665
##########
File path:
hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/ratis/OzoneManagerRatisServer.java
##########
@@ -317,8 +377,18 @@ public static OzoneManagerRatisServer newOMRatisServer(
.build();
List<RaftPeer> raftPeers = new ArrayList<>();
- // Add this Ratis server to the Ratis ring
- raftPeers.add(localRaftPeer);
+
+ // If the OM is started in bootstrap mode, do not add it to the ratis ring.
+ // It will be added later using SetConfiguration from the leader OM.
+ if (isBootstrapping) {
Review comment:
It could possibly cause a different ring to form with the new OMs.
----------------------------------------------------------------
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]
