[GitHub] [ozone] siddhantsangwan commented on pull request #3547: HDDS-6940. EC: Skip the EC container for balancer

[GitBox <gi...@apache.org>]

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: issues-unsubscribe@ozone.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org
For additional commands, e-mail: issues-help@ozone.apache.org