[ 
https://issues.apache.org/jira/browse/HADOOP-8649?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13429429#comment-13429429
 ] 

Daryn Sharp commented on HADOOP-8649:
-------------------------------------

I just eyeballed the fs classes on trunk.  The {{listStatus}} variants all 
depend on the {{listStatus(Path)}} primitive.  {{ChecksumFileSystem}} is 
overriding this to filter out crcs. {{FileSystem#listStatus(Path, PathFilter)}} 
calls {{CheckSumFileSystem#listStatus(Path)}} and then applies the user given 
filter.  It looks like it should already work.  Where exactly is the bug and is 
it only in branch-1?
                
> ChecksumFileSystem should have an overriding implementation of 
> listStatus(Path, PathFilter)
> -------------------------------------------------------------------------------------------
>
>                 Key: HADOOP-8649
>                 URL: https://issues.apache.org/jira/browse/HADOOP-8649
>             Project: Hadoop Common
>          Issue Type: Bug
>            Reporter: Karthik Kambatla
>            Assignee: Karthik Kambatla
>         Attachments: HADOOP-8649_branch1.patch, HADOOP-8649_branch1.patch_v2, 
> HADOOP-8649_branch1.patch_v3
>
>
> Currently, ChecksumFileSystem implements only listStatus(Path). The other 
> form of listStatus(Path, PathFilter) is implemented by parent class 
> FileSystem, and hence doesn't filter out check-sum files.
> The implementation should use a composite filter of passed Filter and the 
> Checksum filter.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to