[GitHub] [hadoop-ozone] linyiqun commented on pull request #1459: HDDS-4297. Allow multiple transactions per container to be sent for deletion by SCM.

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

linyiqun commented on pull request #1459:
URL: https://github.com/apache/hadoop-ozone/pull/1459#issuecomment-708509231


   @lokeshj1703 , seems the original purpose of letting only one transaction per container to delete on datanode is to make container block deletion more balanced. So the containers will have the similar available space for SCM to choose to do block allocation.
   
   But current existed implement is not friendly for the scenario that our deleted blocks focus on the same (or very few)containers in the datanode. And current PR change is helpful on this scenario. I think container block allocation should be random, not sure if this corner case will happened.
   


----------------------------------------------------------------
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.

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



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