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

Harsh J commented on MAPREDUCE-4216:
------------------------------------

Hey Robbie, thanks for the patch and the report!

Do we _have_ to move static methods into the abstract OutputFormat class? Why 
can't it remain in the FileOutputFormat class itself?

I agree MultipleOutputs should allow non FOF-extending classes (though it was 
primarily written for file classes), however why doesn't your first change hunk 
alone suffice for it?
                
> Make MultipleOutputs generic to support non-file output formats
> ---------------------------------------------------------------
>
>                 Key: MAPREDUCE-4216
>                 URL: https://issues.apache.org/jira/browse/MAPREDUCE-4216
>             Project: Hadoop Map/Reduce
>          Issue Type: Improvement
>          Components: mrv2
>    Affects Versions: 1.0.2
>            Reporter: Robbie Strickland
>              Labels: Output
>         Attachments: MAPREDUCE-4216.patch
>
>
> The current MultipleOutputs implementation is tied to FileOutputFormat in 
> such a way that it is not extensible to other types of output. It should be 
> made more generic, such as with an interface that can be implemented for 
> different outputs.

--
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