JacksonYao287 commented on pull request #2808: URL: https://github.com/apache/ozone/pull/2808#issuecomment-971544876
thanks @lokeshj1703 very much for the review, i have updated the patch, please take a look! >Max limits helps in limiting the amount of data replicated from/to dn which I think is important otherwise a dn could be overburdened with too many replications. i do get your point，I have also considered this。but current Max limit is just the same as a default container size(5G), and i think it is too small to support moving more than one container. It depends and is debatable what value to set, so i do not specified it for now. what about 500G by default, or do you have any suggestion? we can set this in the later PR. -- 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]
