Details
-
Task
-
Status: Closed
-
Trivial
-
Resolution: Fixed
-
2.3.0
-
None
-
None
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.