You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@ozone.apache.org by GitBox <gi...@apache.org> on 2021/02/18 01:22:11 UTC

[GitHub] [ozone] amaliujia commented on pull request #1929: HDDS-4811.Use OM style config to construct RaftGroup and initialize Raft Servers

amaliujia commented on pull request #1929:
URL: https://github.com/apache/ozone/pull/1929#issuecomment-780968365


   @bshashikant 
   
   Regrading  the SCMHANodeDetails#174 TODO, I used to think we load Raft config into SCMHANodeDetails directly. however, now the code is refactored as https://github.com/apache/ozone/blob/b60a2306374533a616123d1f9da33e0506bac4d6/hadoop-hdds/server-scm/src/main/java/org/apache/hadoop/hdds/scm/ha/SCMRatisServerImpl.java#L93
   
   So I guess there is no need for the SCMHANodeDetails#174 TODO.
   
   At this moment HDDS-4811 maybe is already just a no-op JIRA 


----------------------------------------------------------------
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:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org
For additional commands, e-mail: issues-help@ozone.apache.org