[ https://issues.apache.org/jira/browse/HADOOP-12977?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15546525#comment-15546525 ]
Chris Nauroth commented on HADOOP-12977: ---------------------------------------- I'm not sure a test for "//" adds much value, considering that when the path string gets passed through {{Path}} / {{URI}}, the consecutive '/' characters will get normalized down to just one '/'. There is no harm in testing it, but I think it just degenerates to the same thing as testing "/". This would really be more of a unit test of {{Path}} behavior than {{FileSystem}} behavior. Actually, I just checked and that's covered by a unit test in {{TestPath#testNormalize}}. > s3a ignores delete("/", true) > ----------------------------- > > Key: HADOOP-12977 > URL: https://issues.apache.org/jira/browse/HADOOP-12977 > Project: Hadoop Common > Issue Type: Sub-task > Components: fs/s3 > Affects Versions: 2.9.0 > Reporter: Steve Loughran > Assignee: Steve Loughran > Priority: Minor > Attachments: HADOOP-12977-001.patch, HADOOP-12977-002.patch, > HADOOP-12977-branch-2-002.patch, HADOOP-12977-branch-2-002.patch > > > if you try to delete the root directory on s3a, you get politely but firmly > told you can't > {code} > 2016-03-30 12:01:44,924 INFO s3a.S3AFileSystem > (S3AFileSystem.java:delete(638)) - s3a cannot delete the root directory > {code} > The semantics of {{rm -rf "/"}} are defined, they are "delete everything > underneath, while preserving the root dir itself". > # s3a needs to support this. > # this skipped through the FS contract tests in > {{AbstractContractRootDirectoryTest}}; the option of whether deleting / works > or not should be made configurable. -- This message was sent by Atlassian JIRA (v6.3.4#6332) --------------------------------------------------------------------- To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org