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

Arpit Agarwal edited comment on HDFS-9788 at 2/10/16 10:35 PM:
---------------------------------------------------------------

Good catch. I don't think the YARN-3330 script would have caught it. We'd need 
to diff proto files across release lines to detect such issues.


was (Author: arpitagarwal):
Good catch. I don't think the YARN-3330 script can have caught it. We'd need to 
diff proto files across release lines to detect such issues.

> Incompatible tag renumbering in HeartbeatResponseProto
> ------------------------------------------------------
>
>                 Key: HDFS-9788
>                 URL: https://issues.apache.org/jira/browse/HDFS-9788
>             Project: Hadoop HDFS
>          Issue Type: Bug
>          Components: rolling upgrades
>    Affects Versions: 2.8.0
>            Reporter: Andrew Wang
>            Assignee: Andrew Wang
>            Priority: Blocker
>         Attachments: HDFS-9788.001.patch
>
>
> The HDFS-9426 patches for branch-2.7 and branch-2 used different tag numbers 
> for {{rollingUpgradeStatusV2}} in HeartbeatResponseProto:
> trunk/branch-2:
> {code}
> message HeartbeatResponseProto {
>   repeated DatanodeCommandProto cmds = 1; // Returned commands can be null
>   required NNHAStatusHeartbeatProto haStatus = 2;
>   optional RollingUpgradeStatusProto rollingUpgradeStatus = 3;
>   optional uint64 fullBlockReportLeaseId = 4 [ default = 0 ];
>   optional RollingUpgradeStatusProto rollingUpgradeStatusV2 = 5;
> }
> {code}
> branch-2.7:
> {code}
> message HeartbeatResponseProto {
>   repeated DatanodeCommandProto cmds = 1; // Returned commands can be null
>   required NNHAStatusHeartbeatProto haStatus = 2;
>   optional RollingUpgradeStatusProto rollingUpgradeStatus = 3;
>   optional RollingUpgradeStatusProto rollingUpgradeStatusV2 = 4;
> }
> {code}
> This breaks rolling upgrade between 2.7 and a future 2.8. We need to renumber 
> the tags to preserve wire compatibility.



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

Reply via email to