[
https://issues.apache.org/jira/browse/MAPREDUCE-6332?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14515860#comment-14515860
]
Vinod Kumar Vavilapalli commented on MAPREDUCE-6332:
----------------------------------------------------
Regarding the patch
- MergeThread.java directly uses the merge-manager impl, that should be fixed
too?
- Add some javadoc to the new methods?
> Add more required API's to MergeManager interface
> --------------------------------------------------
>
> Key: MAPREDUCE-6332
> URL: https://issues.apache.org/jira/browse/MAPREDUCE-6332
> Project: Hadoop Map/Reduce
> Issue Type: Bug
> Affects Versions: 2.5.0, 2.6.0, 2.7.0
> Reporter: Rohith
> Assignee: Rohith
> Attachments: 0001-MAPREDUCE-6332.patch
>
>
> MR provides ability to the user for plugin custom ShuffleConsumerPlugin using
> *mapreduce.job.reduce.shuffle.consumer.plugin.class*. When the user is
> allowed to use this configuration as plugin, user also interest in
> implementing his own MergeManagerImpl.
> But now , user is forced to use MR provided MergeManagerImpl instead of
> custom MergeManagerImpl when user is using shuffle.consumer.plugin class.
> There should be well defined API's in MergeManager that can be used for any
> implementation without much effort to user for custom implementation.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
