hanishakoneru commented on pull request #2491: URL: https://github.com/apache/ozone/pull/2491#issuecomment-963665745
Thanks @bharatviswa504 for the review. I have addressed your comments in the new patch or answered inline. > Just a question, why cant we add this to OMInterServiceProtocol? Any reason to add it as a fresh OMMetadataProtocol? Added a new protocol so that it can be used for removeOM admin command also. RemoveOM would have to be an admin command and not from other OMs. So we need a new protocol for admins to perform operations on the OM. -- 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]
