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

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

Since we know exactly how many volumes in FsDatasetImpl.getStorageReports(..), 
why don't we create an array for the reports in the first place?  I.e.
{code}
    final StorageReport[] reports = new StorageReport[volumes.volumes.size()];
{code}

> Heartbeats from Datandode should include one storage report per storage 
> directory
> ---------------------------------------------------------------------------------
>
>                 Key: HDFS-5377
>                 URL: https://issues.apache.org/jira/browse/HDFS-5377
>             Project: Hadoop HDFS
>          Issue Type: Sub-task
>    Affects Versions: Heterogeneous Storage (HDFS-2832)
>            Reporter: Arpit Agarwal
>            Assignee: Arpit Agarwal
>         Attachments: h5377.10.patch
>
>
> Heartbeats must include one storage report per storage directory, so that 
> usage information for each directory is available to the DataNode for block 
> placement.
> This is a continuation of the work started by HDFS-5134 and HDFS-4988 to 
> split block processing per storage directory.



--
This message was sent by Atlassian JIRA
(v6.1#6144)

Reply via email to