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

Lei (Eddy) Xu commented on HDFS-9004:
-------------------------------------

Hi, [~mingma]

You patch looks very good in general. 

It'd be great to address one small change: is the following change necessary in 
this patch?

{code:title=DFSTestUtil.java}
        1l, 2l, 3l, 4l, 0l, 0l, 0l, 5, 6, "local", adminState,
+       ipAddr + ":" + DFSConfigKeys.DFS_DATANODE_DEFAULT_PORT);
{code}

Will +1 once address the above comments and verify the tests failures are not 
relevant.

> Add upgrade domain to DatanodeInfo
> ----------------------------------
>
>                 Key: HDFS-9004
>                 URL: https://issues.apache.org/jira/browse/HDFS-9004
>             Project: Hadoop HDFS
>          Issue Type: Sub-task
>            Reporter: Ming Ma
>            Assignee: Ming Ma
>         Attachments: HDFS-9004-2.patch, HDFS-9004.patch
>
>
> As part of upgrade domain feature, we first need to add upgrade domain string 
> to {{DatanodeInfo}}. It includes things like:
> * Add a new field to DatanodeInfo.
> * Modify protobuf for DatanodeInfo.
> * Update DatanodeInfo.getDatanodeReport to include upgrade domain.



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

Reply via email to