JacksonYao287 commented on PR #3547: URL: https://github.com/apache/ozone/pull/3547#issuecomment-1170662211
@umamaheswararao there are two steps when balancing container: 1 containerBalancer will select a candidate container from a source over-utilized Dn, and find an under-utilized Dn as a target 2 call legacyRm#move to take this move action. the only difference for EC container in step 1 is the placement policy, this is what is done in [HDDS-6533](https://github.com/apache/ozone/pull/3455), and i think it has nothing to do with RM. we can merge is first. for EC container, we can just skip move in legacy in RM until we complete the RM related work. @siddhantsangwan if container balancer will select an EC container but the move will fail since we skip it now. so theoretically，there might be a scenario that an EC container might always be selected for move ,but move always fail and the utilization of the datanode will never change, which will lead to Infinite loop。 please notice this. we can fix this after RM related work is completed -- 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]
