hadoop-hdfs-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Yongjun Zhang (JIRA)" <j...@apache.org>
Subject [jira] [Updated] (HDFS-6030) to remove a redundant/unused constructor in INode.java
Date Fri, 28 Feb 2014 00:26:19 GMT

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

Yongjun Zhang updated HDFS-6030:
--------------------------------

    Attachment: HDFS-6030.001.patch

> to remove a redundant/unused constructor in INode.java
> ------------------------------------------------------
>
>                 Key: HDFS-6030
>                 URL: https://issues.apache.org/jira/browse/HDFS-6030
>             Project: Hadoop HDFS
>          Issue Type: Task
>            Reporter: Yongjun Zhang
>            Priority: Trivial
>         Attachments: HDFS-6030.001.patch
>
>
> When I look at fix of HDFS-4879, I noticed that class BlocksMapUpdateInfo in INode.java,
 has a constructor that still uses ArrayList<Block> (which HDFS-4879 tries to replace
with ChunkedArrayList for better memory efficiency)
> {code}
>     public static class BlocksMapUpdateInfo { 
>     ...... 
>     public BlocksMapUpdateInfo(List<Block> toDeleteList) {
>       this.toDeleteList = toDeleteList == null ? new ArrayList<Block>()
>           : toDeleteList;
>     }
>     ......
>     }
> {code}
> It turns out that this constructor is not used anywhere. I'm filing this JIRA to remove
it.



--
This message was sent by Atlassian JIRA
(v6.1.5#6160)

Mime
View raw message