[HDFS-6030] Remove an unused constructor in INode.java

• Type: Task
• Status: Closed
• Priority: Trivial
• Resolution: Fixed
• Affects Version/s: 2.3.0
• Fix Version/s: 2.4.0
• Assignee: Yongjun Zhang
• Reporter: Yongjun Zhang
• Created: 27/Feb/14 22:21
• Updated: 10/Apr/14 13:11
• Resolved: 28/Feb/14 18:53

Description

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)

    public static class BlocksMapUpdateInfo { 
    ...... 
    public BlocksMapUpdateInfo(List<Block> toDeleteList) {
      this.toDeleteList = toDeleteList == null ? new ArrayList<Block>()
          : toDeleteList;
    }
    ......
    }

It turns out that this constructor is not used anywhere (thanks cmccabe for pointing this out). I'm filing this JIRA to remove it.