hevinhsu commented on PR #10598: URL: https://github.com/apache/ozone/pull/10598#issuecomment-4829301038
Hi @ivandika3, thanks for your suggestion. I've added `TestRackAwarePlacement` to cover the rack-aware placement scenarios. Based on my understanding, the rack-aware placement policies are covered as follows: | Area | Coverage | | --- | --- | | Pipeline placement | Verified rack-aware pipeline placement. From what I found, rack awareness is handled by `PipelinePlacementPolicy`, which is the default implementation. | | Container placement | Verified closed container re-replication with both `SCMContainerPlacementRackAware` and `SCMContainerPlacementRackScatter`. | If I misunderstood the placement policies or missed another rack-aware policy that should be covered, please let me know. -- 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]
