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

Chris Nauroth commented on HDFS-7748:
-------------------------------------

Thanks for the revised patch, Anu.  I have just one more minor nitpick.  
hdfs-site.xml is still in the patch with a whitespace change.  Let's remove it 
from the patch completely.  With that, all of my feedback has been addressed, 
and I'm +1 pending another Jenkins run.  I'll wait for [~wheat9] to take 
another look before committing.

> 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, hdfs-7748.002.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