[
https://issues.apache.org/jira/browse/TEZ-1386?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14089575#comment-14089575
]
Siddharth Seth commented on TEZ-1386:
-------------------------------------
I don't really think this is a big concern - leaving the InputFormat at what it
was - since MRInput is how this is processed. Usually, a user should see the
InputFormat that they setup.
The change to the proto etc isn't really necessary for the main intent of the
jira. Just think that's a better way to get this done, especially if we're
changing with the configuration specified by the user. In any case, I'll change
this bit back - since it isn't critical for the patch, and this needs to go in
to continue on TEZ-1347.
> Users should not need to setup TezGroupedInputFormat to enable grouping
> -----------------------------------------------------------------------
>
> Key: TEZ-1386
> URL: https://issues.apache.org/jira/browse/TEZ-1386
> Project: Apache Tez
> Issue Type: Improvement
> Reporter: Siddharth Seth
> Assignee: Siddharth Seth
> Attachments: TEZ-1386.1.txt
>
>
> To enable grouping via Tez, users should not need to change the underlying
> InputFormat. A simple enable / disable option should be sufficient.
> MRInputConfigurer does this.
> Many of the methods in MRHelpers, however, require an InputFormat to be
> specified. The main objective of this JIRA is to get rid of this requirement
> in favor of a simple enableGrouping flag.
> Marking this as a blocker for TEZ-1347, since it should simplify the set of
> APIs required. Also, making all the changes in TEZ-1347 would just lead to a
> very large patch.
--
This message was sent by Atlassian JIRA
(v6.2#6252)
