[
https://issues.apache.org/jira/browse/HDDS-16247?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=18110355#comment-18110355
]
Chia-Chuan Ho commented on HDDS-16247:
--------------------------------------
Hi [~szetszwo] , the PR for HDDS-16247 is now ready.
It implements the agreed approach for StorageContainerLocationProtocol and
includes coverage for the relevant RPC identity, authorization, and HA paths.
Could you please take a look when you have a chance?
Thank you!
> Separate StorageContainerLocationProtocol RPC identity from its internal Java
> interface
> ---------------------------------------------------------------------------------------
>
> Key: HDDS-16247
> URL: https://issues.apache.org/jira/browse/HDDS-16247
> Project: Apache Ozone
> Issue Type: Sub-task
> Reporter: Chia-Chuan Ho
> Assignee: Chia-Chuan Ho
> Priority: Major
> Labels: pull-request-available
>
> Follow-up to HDDS-16242.
>
> StorageContainerLocationProtocol is not the wire protocol. The actual RPC
> protocol is defined by StorageContainerLocationProtocolService and exposed
> through StorageContainerLocationProtocolPB.
>
> Move the existing Java methods from StorageContainerLocationProtocol to a new
> internal interface, such as StorageContainerLocationInternalInterface.
>
> Retain StorageContainerLocationProtocol as an empty class with the same fully
> qualified class name because it is still used as the Hadoop RPC protocol
> identity and service-authorization ACL key.
>
> Update internal implementations and callers to use the new internal
> interface. Keep StorageContainerLocationProtocolPB, its @ProtocolInfo
> protocolName, Protobuf definitions, translators, and wire protocol unchanged.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
