[
https://issues.apache.org/jira/browse/HDFS-11103?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15641052#comment-15641052
]
Xiaoyu Yao commented on HDFS-11103:
-----------------------------------
Thanks [~anu] for working on this. The patch looks good to me overall. Just
some NITS:
*ContainerLocationManagerImpl.java*
1. Line 55
{code}
volumePaths = dataDirs.toArray(new Path[0]); =>dataDirs.toArray(new
Path[dataDirs.size()])
{code}
Allocate array to the right size to avoid unnecessary reallocate.
2. Can we add some comments for {{volumePaths}} and {{locations}} for example,
one for container data and one for container metadata?
*ContainerManagerImpl.java*
3. Extra blank line 95
4. Do you consider maintaining a list of StorageLocation instead a list of
Paths inside ContainerLocationManagerImpl so that we can leverage the storage
type information from StorageLocation for use case as documented in
ContainerLocationMangerImpl.java below. If this causes too much churn across
the code base. It is OK to leave this for later with some additional TODOs.
{code}
* For example : A user could map all container files to a SSD but leave
data/metadata on bunch of other disks.
{code}
> Ozone: Cleanup some dependencies
> --------------------------------
>
> Key: HDFS-11103
> URL: https://issues.apache.org/jira/browse/HDFS-11103
> Project: Hadoop HDFS
> Issue Type: Sub-task
> Components: ozone
> Affects Versions: HDFS-7240
> Reporter: Anu Engineer
> Assignee: Anu Engineer
> Priority: Trivial
> Fix For: HDFS-7240
>
> Attachments: HDFS-11103-HDFS-7240.001.patch,
> HDFS-11103-HDFS-7240.002.patch
>
>
> Cleanup some unwanted dependencies.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
