[ 
https://issues.apache.org/jira/browse/HDDS-3621?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Nanda kumar resolved HDDS-3621.
-------------------------------
    Fix Version/s: 0.6.0
       Resolution: Fixed

> Separate client/server/admin proto files of HDDS to separated subprojects 
> --------------------------------------------------------------------------
>
>                 Key: HDDS-3621
>                 URL: https://issues.apache.org/jira/browse/HDDS-3621
>             Project: Hadoop Distributed Data Store
>          Issue Type: Sub-task
>            Reporter: Marton Elek
>            Assignee: Marton Elek
>            Priority: Major
>              Labels: pull-request-available
>             Fix For: 0.6.0
>
>
> As described in the parent Jira admin/client/server protocols need different 
> compatibility guarantees. It's better to separate them to separated maven 
> project to make it easy to follow the changes.
> I propose to create 3 new projects
> hadoop-hdds/interface-client
> hadoop-hdds/interface-server
> hadoop-hdds/interface-admin
> I called it -interface instead of -proto because we can include some basic 
> Java classes not just proto files (for example utilities to 
> serialize/deserialize proto files)
> This first patch will include only the move without refactoring any RPC. 
> While the new proto file names represent the proposed naming convention 
> (Datanode/Scm + Server/Client/Admin + additional postfix) the name of the 
> generated classes and the name of the RPC interfaces (Client/Server 
> translator) not yet renamed to make the patch small.
> Also: some methods should be moved between admin/server but it can be done in 
> separated issue to make it easier to follow the change.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

---------------------------------------------------------------------
To unsubscribe, e-mail: ozone-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: ozone-issues-h...@hadoop.apache.org

Reply via email to