sodonnel opened a new pull request #984: HDDS-1674 Make ScmBlockLocationProtocol message type based URL: https://github.com/apache/hadoop/pull/984 This PR is a first attempt at refactoring the ScmBlockLocationProtocol using a single message type as is used in the OzoneManagerProtocol. In this change, the new message wraps the existing messages and the translator classes simply wrap or unwrap it. Only TraceID has been moved to the wrapper message - Moving error handling and error codes to the wrapper will be done in a separate change. Before this can be merged we still need to determine if the clientId should be present in the ScmBlockLocationProtocol. It is in OzoneManagerProtocol and has been replicated here for now, but it can be removed if needed.
---------------------------------------------------------------- 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: [email protected] With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
