errose28 opened a new pull request, #10936: URL: https://github.com/apache/ozone/pull/10936
## What changes were proposed in this pull request? In the new versioning framework, the `ClientVersion` class should be used within the java code, and only delegate to `ClientVersion#serialize/deserialize` at the translation layers of the client and server. This PR updates existing usage of `ClientVersion` to follow this practice. The main change was in the protobuf translators for `DatanodeDetails`, which now take a `ClientVersion` and call `serialize` since those methods are already in charge of serialization. This creates a ripple effect to classes that use `DatanodeDetails` which have been updated to pass it a `ClientVersion` instead of serializing the version themselves before passing it in. Many files were touched, but it is a simple substitution that does not amount in a large number of lines changed. ## What is the link to the Apache JIRA HDDS-14753 ## How was this patch tested? No functional change. All existing tests should pass. Changes do not affect protobuf/disk/wire protocols -- 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]
