[ 
https://issues.apache.org/jira/browse/HDDS-39?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16471448#comment-16471448
 ] 

Mukul Kumar Singh commented on HDDS-39:
---------------------------------------

[~xyao] and [~szetszwo] Please have a look at v3 patch.

This patch compiles DatanodeContainerProtocol.proto using proto3 compiler. This 
patch will further avoid buffer copy in XceiverClientRatis and 
ContainerStateMachine.
This change will also enables native client integration for Ozone using grpc 
C++ apis.

Most of the changes are import changes, actual changes are limited to 
{{XceiverClientRatis}}, {{ContainerStateMachine}} and 
{{hadoop-hdds/common/pom.xml}}.

> Ozone: Compile Ozone/HDFS/Cblock protobuf files with proto3 compiler using 
> maven protoc plugin
> ----------------------------------------------------------------------------------------------
>
>                 Key: HDDS-39
>                 URL: https://issues.apache.org/jira/browse/HDDS-39
>             Project: Hadoop Distributed Data Store
>          Issue Type: Improvement
>          Components: Native
>    Affects Versions: 0.2.1
>            Reporter: Mukul Kumar Singh
>            Assignee: Mukul Kumar Singh
>            Priority: Major
>             Fix For: 0.2.1
>
>         Attachments: HDDS-39.003.patch, HDFS-13389-HDFS-7240.001.patch, 
> HDFS-13389-HDFS-7240.002.patch
>
>
> Currently all the Ozone/HDFS/Cblock proto files are compiled using proto 2.5, 
> this can be changed to use proto3 compiler.
> This change will help in performance improvement as well because currently in 
> the client path, the xceiver client ratis converts proto2 classes to proto3 
> using byte string manipulation.
> Please note that for rest of hadoop (except Ozone/Cblock/HDSL), the protoc 
> version will still remain 2.5 as this proto compilation will be done through 
> the following plugin. 
> https://www.xolstice.org/protobuf-maven-plugin/



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to