siddhantsangwan opened a new pull request, #3485: URL: https://github.com/apache/ozone/pull/3485
## What changes were proposed in this pull request? The sourceToTargetMap, being a map, will not support multiple source to target mappings with the same source. Since ContainerBalancer now allows a source to be matched with multiple targets, this data structure is no longer suitable. This map is used in multiple unit tests and in the counting of Datanodes involved in an iteration. Since a particular container can be selected only once in an iteration, this PR proposes using ContainerID as the primary key to track Container->Source DN and Container->Target DN move pairings. ## What is the link to the Apache JIRA https://issues.apache.org/jira/browse/HDDS-6243 ## How was this patch tested? Modified the existing UTs in `TestContainerBalancer`. -- 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]
