sodonnel commented on code in PR #8942: URL: https://github.com/apache/ozone/pull/8942#discussion_r2352103680
########## hadoop-hdds/docs/content/feature/Topology.md: ########## @@ -173,6 +191,8 @@ This directs clients (replicated data) to read from topologically closest DataNo * **Accurate Topology:** Maintain an accurate, up-to-date topology map (static or dynamic script); this is foundational. * **Pipeline Creation:** For production environments, use the default `PipelinePlacementPolicy` for `ozone.scm.pipeline.placement.impl` to ensure both rack fault tolerance and pipeline load balancing. * **Pipeline Selection:** The default `RandomPipelineChoosePolicy` for `hdds.scm.pipeline.choose.policy.impl` is suitable for general load balancing. +* **Replicated (RATIS) Containers:** For production rack fault tolerance, use `SCMContainerPlacementRackAware` (mindful of its single-layer topology limitation) or `SCMContainerPlacementCapacity` (verify rack interaction) over `SCMContainerPlacementRandom`. Review Comment: I am not sure about this line, especially: > or `SCMContainerPlacementCapacity` (verify rack interaction) I guess capacity is not rack aware, so you can either have rack awareness, or capacity, but not both with our current code / options. -- 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: issues-unsubscr...@ozone.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@ozone.apache.org For additional commands, e-mail: issues-h...@ozone.apache.org
