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

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

I'm just generally concerned about the implications of stacking filesystems.  
Ie. a {{FilterFileSystem}} over a {{ChRootedFileSystem}} over a 
{{FilterFileSystem}}, etc.  I'm not sure it's a problem, but you should make 
sure there are tests that prove the stacking works.

I conceptually like the approach suggested.  Throw something up and let's see 
how it looks!
                
> ChecksumFileSystem should have an overriding implementation of 
> listStatus(Path, PathFilter) for improved performance
> --------------------------------------------------------------------------------------------------------------------
>
>                 Key: HADOOP-8649
>                 URL: https://issues.apache.org/jira/browse/HADOOP-8649
>             Project: Hadoop Common
>          Issue Type: Improvement
>    Affects Versions: 1.0.3, 2.0.0-alpha
>            Reporter: Karthik Kambatla
>            Assignee: Karthik Kambatla
>         Attachments: branch1-HADOOP-8649.patch, branch1-HADOOP-8649.patch, 
> HADOOP-8649_branch1.patch, HADOOP-8649_branch1.patch, 
> HADOOP-8649_branch1.patch_v2, HADOOP-8649_branch1.patch_v3, 
> TestChecksumFileSystemOnDFS.java, trunk-HADOOP-8649.patch, 
> trunk-HADOOP-8649.patch
>
>
> Currently, ChecksumFileSystem implements only listStatus(Path). 
> The other form of listStatus(Path, customFilter) results in parsing the list 
> twice to apply each of the filters - custom and checksum filter.
> By using a composite filter instead, we limit the parsing to once.

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