hadoop-common-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Rodrigo Schmidt (JIRA)" <j...@apache.org>
Subject [jira] Commented: (HADOOP-6645) Bugs on listStatus for HarFileSystem
Date Mon, 22 Mar 2010 07:20:27 GMT

    [ 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.


Mime
View raw message