siddhantsangwan commented on PR #3547: URL: https://github.com/apache/ozone/pull/3547#issuecomment-1180039478
Thanks for the reviews. > 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 To avoid this scenario, we can merge this PR to exclude EC containers from balancing for now. We can then proceed with https://github.com/apache/ozone/pull/3455 and finally include EC containers (by undoing the code in this PR) once steps 1 and 2 are complete. What do you all think? -- 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]
