[ https://issues.apache.org/jira/browse/HDFS-2011?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13055660#comment-13055660 ]
Ravi Prakash commented on HDFS-2011: ------------------------------------ Thanks Matt, Incorporated all your comments :) {quote} 9. In "assertTrue("List of storage directories didn't have storageDirToCheck."...), did you intend to iterate over all elements in the list? You go to the trouble of creating an iterator, and then only use the first element. {quote} I meant to get the first element of the Collection (since that's what nnStorage.getEditsDirectories() returns me). > Removal and restoration of storage directories on checkpointing failure > doesn't work properly > --------------------------------------------------------------------------------------------- > > Key: HDFS-2011 > URL: https://issues.apache.org/jira/browse/HDFS-2011 > Project: Hadoop HDFS > Issue Type: Bug > Components: name-node > Affects Versions: 0.23.0 > Reporter: Ravi Prakash > Assignee: Ravi Prakash > Attachments: HDFS-2011.3.patch, HDFS-2011.4.patch, HDFS-2011.5.patch, > HDFS-2011.6.patch, HDFS-2011.patch, HDFS-2011.patch, HDFS-2011.patch > > > Removal and restoration of storage directories on checkpointing failure > doesn't work properly. Sometimes it throws a NullPointerException and > sometimes it doesn't take off a failed storage directory -- This message is automatically generated by JIRA. For more information on JIRA, see: http://www.atlassian.com/software/jira