jojochuang commented on code in PR #8942: URL: https://github.com/apache/ozone/pull/8942#discussion_r2364443855
########## 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: Oh yeah you're right. Missed it. SCMContainerPlacementCapacity does not take into account topology. -- 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]
