[ 
https://issues.apache.org/jira/browse/HADOOP-8649?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Karthik Kambatla updated HADOOP-8649:
-------------------------------------

    Attachment: TestChecksumFileSystemOnDFS.java

Sorry for the false alarm. As per Daryn's suggestion, I wrote a test to check 
the same that I am uploading here.

Daryn's description of the flow is right, and there is no bug. Sorry again.

Also, as Daryn commented, using a composite fiter would improve the performance.

I ll update the description of the JIRA to reflect the same and upload patches 
for branch-1 and test including this test.

Thanks again for your thorough review, Daryn.
                
> 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, TestChecksumFileSystemOnDFS.java
>
>
> 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