[ 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