[
https://issues.apache.org/jira/browse/HADOOP-6645?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12848047#action_12848047
]
Rodrigo Schmidt commented on HADOOP-6645:
-----------------------------------------
Thanks for looking at it, Mahadev!
If we don't add the separator, the selection if will pass for /fooo/bar2 when
we ask to list the contents of /foo in the example I gave in the JIRA
description. Adding Path.SEPARATOR makes sure that we only select children of
the right parent, and not children of a parent that is an extension of the
right one.
The current unit tests don't cover this case... However, unit tests are in the
MAPREDUCE project and I can't add them in HADOOP patches like this one. To
avoid errors, we have to commit this patch first and then add new unit tests to
MAPREDUCE. :-(
> Bugs on listStatus for HarFileSystem
> ------------------------------------
>
> Key: HADOOP-6645
> URL: https://issues.apache.org/jira/browse/HADOOP-6645
> Project: Hadoop Common
> Issue Type: Bug
> Affects Versions: 0.22.0
> Reporter: Rodrigo Schmidt
> Assignee: Rodrigo Schmidt
> Fix For: 0.22.0
>
> Attachments: HADOOP-6645.patch
>
>
> Two bugs on listStatus for HarFileSystem:
> 1) consider the following directory tree inside a hadoop archive
> /foo
> /foo/bar1
> /fooo
> /fooo/bar2
> In this case, listStatus(new Path("/foo")) will include /fooo/bar2 because
> fileStatusesInIndex is testing a prefix.
> 2) HADOOP-6591 didn't take into consideration method fileStatusesInIndex(),
> and archives v2 return empty results for listStatus()
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
