[
https://issues.apache.org/jira/browse/TEZ-1386?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14089583#comment-14089583
]
Bikas Saha commented on TEZ-1386:
---------------------------------
The proto change is actually good. All I want to ensure is that post grouping,
things remain consistent for the downstream consumers without needing to
special case anything. This would also allow us to make
MRHelpers.createOldFormatSplitFromUserPayload() private. Otherwise we would
need to make it public because a downstream consumer would need that helper
method to differentiate the grouping vs non-grouping case.
> 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)
