You are viewing a plain text version of this content. The canonical link for it is here.
Posted to common-issues@hadoop.apache.org by "Robert Joseph Evans (JIRA)" <ji...@apache.org> on 2012/11/01 17:35:13 UTC

[jira] [Commented] (HADOOP-8992) Enhance unit-test coverage of class HarFileSystem

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

Robert Joseph Evans commented on HADOOP-8992:
---------------------------------------------

Thanks for the changes.  My only other comment right now is that it might be nice to combine this test with TestHarFileSystem.java  They are both trying to accomplish the same thing and having two files seems a little unnecessary.
                
> Enhance unit-test coverage of class HarFileSystem
> -------------------------------------------------
>
>                 Key: HADOOP-8992
>                 URL: https://issues.apache.org/jira/browse/HADOOP-8992
>             Project: Hadoop Common
>          Issue Type: Improvement
>            Reporter: Ivan A. Veselovsky
>            Assignee: Ivan A. Veselovsky
>            Priority: Minor
>         Attachments: HADOOP-8992-branch-0.23--a.patch, HADOOP-8992-branch-0.23--b.patch, HADOOP-8992-branch-2--a.patch, HADOOP-8992-branch-2--b.patch
>
>
> New unit test TestHarFileSystem2 provided in order to enhance coverage of class HarFileSystem.
> Also some unused methods deleted from class HarFileSystem.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira