JacksonYao287 edited a comment on pull request #2808: URL: https://github.com/apache/ozone/pull/2808#issuecomment-969835559
@lokeshj1703 @siddhantsangwan i have refactored the patch according to the comments, please take a look. if this looks good to you , i will improve the Code comments in a new commit. >in some case , one container may be moved from a withinThresholdUtilized node to another withinThresholdUtilized node i will create a separate patch to add withinThreshold nodes back into candidate target and source datanodes, and fix the potential bug in the newly refactored code. i think after refactoring, we can do this more gracefully. -- 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]
