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

Haohui Mai commented on HDFS-7748:
----------------------------------

bq. Shall we keep the same values here in the new protobuf enum

Yes. Please keep the enum matching the RFC.

{code}
+  optional ECN ecn = 4 ;
{code}

Let's rename the field into {{header}} and make it a {{int64}} field. That way 
allows us to add more information in the future while maintaining both backward 
and forward compatibility.

> Separate ECN flags from the Status in the DataTransferPipelineAck
> -----------------------------------------------------------------
>
>                 Key: HDFS-7748
>                 URL: https://issues.apache.org/jira/browse/HDFS-7748
>             Project: Hadoop HDFS
>          Issue Type: Bug
>            Reporter: Haohui Mai
>            Assignee: Anu Engineer
>            Priority: Blocker
>         Attachments: hdfs-7748.001.patch
>
>
> Prior to the discussions on HDFS-7270, the old clients might fail to talk to 
> the newer server when ECN is turned on. This jira proposes to separate the 
> ECN flags in a separate protobuf field to make the ack compatible on both 
> versions.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to