You are viewing a plain text version of this content. The canonical link for it is here.

Posted to hdfs-dev@hadoop.apache.org by "Arpit Agarwal (JIRA)" <ji...@apache.org> on 2013/11/22 21:15:35 UTC

[jira] [Resolved] (HDFS-5542) Fix TODO and clean up the code in HDFS-2832.

     [ https://issues.apache.org/jira/browse/HDFS-5542?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Arpit Agarwal resolved HDFS-5542.
---------------------------------

       Resolution: Fixed
    Fix Version/s: Heterogeneous Storage (HDFS-2832)
     Hadoop Flags: Reviewed

+1 for the patch, I committed it to branch HDFS-2832.

Thanks a lot for taking care of the TODOs including many of mine!

> Fix TODO and clean up the code in HDFS-2832.
> --------------------------------------------
>
>                 Key: HDFS-5542
>                 URL: https://issues.apache.org/jira/browse/HDFS-5542
>             Project: Hadoop HDFS
>          Issue Type: Sub-task
>            Reporter: Tsz Wo (Nicholas), SZE
>            Assignee: Tsz Wo (Nicholas), SZE
>            Priority: Minor
>             Fix For: Heterogeneous Storage (HDFS-2832)
>
>         Attachments: h5542_20131121.patch, h5542_20131122.patch
>
>
> - Fix TODOs.
> - Remove unused code.
> - Reduce visibility (e.g. change public to package private.)
> - Simplify the code if possible.
> - Fix comments and javadoc.



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