[
https://issues.apache.org/jira/browse/FALCON-312?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13897032#comment-13897032
]
Venkatesh Seetharam commented on FALCON-312:
--------------------------------------------
The tests work fine with out the fs.delete. Have a patch that removes this and
adds safety clause in
org.apache.falcon.hadoop.JailedFileSystem#delete(org.apache.hadoop.fs.Path,
boolean) as [~sriksun] suggested.
{code}
Path localPath = toLocalPath(f);
if (localPath.toUri().getPath().trim().equals("/")) {
throw new AssertionError("Attempting to delete root " + localPath);
}
return localFS.delete(localPath, recursive);
{code}
> Falcon LogCleanupServiceTest seems to clean up root "/"
> -------------------------------------------------------
>
> Key: FALCON-312
> URL: https://issues.apache.org/jira/browse/FALCON-312
> Project: Falcon
> Issue Type: Bug
> Components: common
> Affects Versions: 0.5
> Reporter: Venkatesh Seetharam
> Priority: Blocker
>
> I have run into this twice. I somehow thought it was osx mavericks the first
> time but the second time happened on a new shiny laptop. The behavior was
> repeatable. When I execute {code}mvn verify -P hadoop-2,test-patch{code},
> this test executes new Path("/") and deletes it.
> I took a look at org.apache.falcon.hadoop.JailedFileSystem and it looks fine.
> I suspect the combination of org.apache.falcon.hadoop.JailedFileSystem and
> hadoop-2. Hadoop-1 works just fine.
> [~sriksun], [~shwethags], [~shaik.idris], can you folks look into this as a
> high priority. Thanks!
--
This message was sent by Atlassian JIRA
(v6.1.5#6160)
