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

Bruno P. Kinoshita commented on MAPREDUCE-5911:
-----------------------------------------------

Hmm, the TestTeraSort is using JUnit 3 and is marked with the @Ignore 
annotation. 

Should we add another class for this test, remove the @Ignore annotation and 
fix the tests, or justify not having a test case for this case? 

Thanks
Bruno

> Terasort TeraOutputFormat does not check for output directory existance
> -----------------------------------------------------------------------
>
>                 Key: MAPREDUCE-5911
>                 URL: https://issues.apache.org/jira/browse/MAPREDUCE-5911
>             Project: Hadoop Map/Reduce
>          Issue Type: Bug
>          Components: examples
>            Reporter: Ivan Mitic
>            Assignee: Bruno P. Kinoshita
>            Priority: Minor
>         Attachments: HADOOP-5911.patch
>
>
> The enforcement that the directory must not yet exist is implemented in 
> {{FileOutputFormat#checkOutputSpecs}} by throwing 
> {{FileAlreadyExistsException}}.  However, terasort uses a specialized output 
> format, {{TeraOutputFormat}}, which is a subclass of {{FileOutputFormat}}.  
> The subclass overrides {{checkOutputSpecs}}, but does not re-implement the 
> existence check and throw {{FileAlreadyExistsException}}.



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

Reply via email to