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

Dustin Cote commented on HADOOP-12564:
--------------------------------------

[~ajisakaa] thanks for the review!  I removed the constructors.  Only thing I 
can't find is the method 
{code}
public void testInitZeroLengthSequenceFile() throws IOException {
{code}

That does exist in my version of TestSequenceFile.java, but maybe I need to 
pull in a new version.  I'll double check that tomorrow, but I'd appreciate if 
you can check to make sure that's the right method I missed tagging.

[~ozawa] I think I've addressed all your comments, so once I get confirmation 
from [~ajisakaa] on the last thing above, I'll add the new patch.  [~ajisakaa] 
please let me know where I can find that method.  Thanks guys!



> Upgrade JUnit3 TestCase to JUnit 4 for tests of org.apache.hadoop.io package
> ----------------------------------------------------------------------------
>
>                 Key: HADOOP-12564
>                 URL: https://issues.apache.org/jira/browse/HADOOP-12564
>             Project: Hadoop Common
>          Issue Type: Test
>          Components: test
>            Reporter: Dustin Cote
>            Assignee: Dustin Cote
>            Priority: Trivial
>         Attachments: MAPREDUCE-6505-1.patch, MAPREDUCE-6505-2.patch, 
> MAPREDUCE-6505-3.patch, MAPREDUCE-6505-4.patch, MAPREDUCE-6505-5.patch
>
>
> Migrating just the io test cases 



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to