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

Tsz Wo (Nicholas), SZE commented on HDFS-5009:
----------------------------------------------

Junping, it is great that you could review the patch.  I started working on it.

----
BTW, I think LocatedBlock should has a DatanodeStorage[] field, where 
DatanodeStorage contains storageID, state and storageType.  If we put 
DatanodeStorage in LocatedBlock, we have to move it from the 
hdfs.server.protocol package to the hdfs.protocol package since LocatedBlock is 
in hdfs.protocol.  Sounds good?
                
> NameNode should expose storage information in the LocatedBlock
> --------------------------------------------------------------
>
>                 Key: HDFS-5009
>                 URL: https://issues.apache.org/jira/browse/HDFS-5009
>             Project: Hadoop HDFS
>          Issue Type: Sub-task
>          Components: namenode
>    Affects Versions: 3.0.0
>            Reporter: Arpit Agarwal
>            Assignee: Tsz Wo (Nicholas), SZE
>
> The storage type is optionally reported by the DataNode along with block 
> report. The NameNode should make this information available to clients via 
> LocatedBlock.
> Also, LocatedBlock is used in reportBadBlocks(..).  We should add storage ID 
> to LocatedBlock.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to