[
https://issues.apache.org/jira/browse/TEZ-1347?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Siddharth Seth updated TEZ-1347:
--------------------------------
Attachment: TEZ-1347.2.txt
Updated patch with comments addressed.
Renamed TezAPIHelpers to TezUtils and TezUtils to TezUtilsInternal. No
consolidation of the various TezUtils in this patch though.
Fixed javadoc, removed Hive/Pig LimitedPrivate. Have removed the unstable from
the API though.
Removed the numReducers check, and the associated fields. For YARNRunner, this
check already runs in the JobClient - so nothing required there.
> Consolidate MRHelpers
> ---------------------
>
> Key: TEZ-1347
> URL: https://issues.apache.org/jira/browse/TEZ-1347
> Project: Apache Tez
> Issue Type: Sub-task
> Reporter: Siddharth Seth
> Assignee: Siddharth Seth
> Priority: Blocker
> Attachments: TEZ-1347-initial-review.txt, TEZ-1347.1.txt,
> TEZ-1347.2.txt
>
>
> - Remove methods which don't belong in MRHelpers and potentially move them to
> TezHelpers.
> - Get rid of methods which we don't expect/want users to use.
> - Get rid of multiple variants of the same method, if these exist.
> - Investigate other cleanup in MRHelpers.
--
This message was sent by Atlassian JIRA
(v6.2#6252)
