fapifta commented on code in PR #4138:
URL: https://github.com/apache/ozone/pull/4138#discussion_r1067460103
##########
hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/OzoneManager.java:
##########
@@ -454,6 +454,11 @@ private OzoneManager(OzoneConfiguration conf,
StartupOption startupOption)
this.omNodeDetails = omhaNodeDetails.getLocalNodeDetails();
omStorage = new OMStorage(conf);
+ // validate if the stored one is the same we get in the OMNodeDetails
+ // if not, fails the start. If the VERSION file does not contain the node
+ // id yet, it stores it to the VERSION file and skips validation for the
+ // that case.
Review Comment:
Based on the previous comment where you seemed to agree that the method
apidoc is enough, I removed this part, and extended the apidoc a bit.
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
