neils-dev opened a new pull request #3074: URL: https://github.com/apache/ozone/pull/3074
## What changes were proposed in this pull request? Necessary due to CI workflow issue, [HDDS-6239](https://issues.apache.org/jira/browse/HDDS-6239), impacting feature branch green builds. This merge also requires partial changes that were initially in PR for [HDDS-5545](https://issues.apache.org/jira/browse/HDDS-5545) to be included. Namely, the patch to allow switching between OmTransports, `GrpcOmTransport` and `Hadoop3OmTransport` through ozone configuration in the `OmConfigKeys`. The current merge from the Master includes changes to interfaces for the s3 gateway client producer, through the `OzoneClientCache`, that removes the need for the omServiceId instead it uses the OmUtils to get the omServiceId through the ozone configuration file. The master merge also includes implementation and tests for handling ozone federation configurations with multiple omServiceIds. In the merged master, unit tests for the s3 gateway were added to test multiple omServiceIds. These new unit tests require changes to the current s3g feature branch that were included in the PR for HDDS-5545. For this those changes that allowed switching omTransports through configuration for the s3 gateway are put into this PR, thereby splitting the PR for 5545. These patches applied to this merge passes all s3 gateway unit tests. These patches include: Changes to the `OmTransportFactory.java` (switch between service provider and configuration OmTransports), `OmConfigKeys.java` (set the OmTransport to be used by the s3 gateway), and the unit test `TestGrpcOmTransport.java`. With this patch, by default the s3 gateway uses hadoop rpc and the Hadoop3OmTransport connection between the s3g and the om. The configuration variable affecting this is `ozone.om.transport.class` set to `org.apache.hadoop.ozone.om.protocolPB.Hadoop3OmTransportFactory` for hrpc (default) and set to `org.apache.hadoop.ozone.om.protocolPB.GrpcOmTransportFactory `for s3g Grpc connection. ## What is the link to the Apache JIRA https://issues.apache.org/jira/browse/HDDS-6303 ## How was this patch tested? S3 gateway unit tests, PASSED all tests. MR acceptance tests - PASSED -- 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]
