ivandika3 commented on code in PR #9637:
URL: https://github.com/apache/ozone/pull/9637#discussion_r2693091574


##########
hadoop-hdds/docs/content/feature/multi-raft-support.md:
##########
@@ -105,6 +105,22 @@ Ratis handles concurrent logs per node.
   The Ratis pipelines will be distributed accordingly.
   - Be cautious with very high pipeline counts due to memory/CPU overhead.
 
+## Advanced Ratis Configuration
+
+Any Ratis configuration properties can be set using a corresponding prefix.
+The following table shows the prefixes in each Ozone component.
+
+| Ozone Components                | Configuration Prefixes  |
+|---------------------------------|-------------------------|
+| Ozone Manager (OM)              | `ozone.om.ha`           |
+| Storage Container Manager (SCM) | `ozone.scm.ha`          |
+| Recon                           | `ozone.recon.scmconfig` |

Review Comment:
   From what I see, "ozone.recon.scmconfig" is only used for 
`ReconStorageContainerManagerFacade#getReconScmConfiguration`. I couldn't find 
any `RaftProperties#set` for Recon.
   
   I also don't think Recon uses Ratis.



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

Reply via email to