fapifta commented on pull request #2910: URL: https://github.com/apache/ozone/pull/2910#issuecomment-993092861
Hi @umamaheswararao, thank you for your great work on this PR. I think the logic itself is good, and I don't see any big problems, but I have a few things I would like you to consider and if you agree change some parts of the code based on that. Please find my notes inline. -- 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]
