[ https://issues.apache.org/jira/browse/HDFS-13054?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16336335#comment-16336335 ]
Arpit Agarwal commented on HDFS-13054: -------------------------------------- One more suggestion, you can add a fail(...) call immediately after the fs.delete call in the test case. {code} fs.delete(new Path("/tmp/nonEmptyDir"), false); <<Add fail here>> {code} So if no exception is thrown then the test case will fail. +1 on the v1 patch either way, pending Jenkins. > Handling PathIsNotEmptyDirectoryException in DFSClient delete call > ------------------------------------------------------------------ > > Key: HDFS-13054 > URL: https://issues.apache.org/jira/browse/HDFS-13054 > Project: Hadoop HDFS > Issue Type: Bug > Components: hdfs-client > Reporter: Nanda kumar > Assignee: Nanda kumar > Priority: Major > Attachments: HDFS-13054.000.patch, HDFS-13054.001.patch > > > In {{DFSClient#delete}} call, if we get > {{RemoteException(PathIsNotEmptyDirectoryException)}} we should unwrap and > throw {{PathIsNotEmptyDirectoryException}} to the caller. -- This message was sent by Atlassian JIRA (v7.6.3#76005) --------------------------------------------------------------------- To unsubscribe, e-mail: hdfs-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: hdfs-issues-h...@hadoop.apache.org