[
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.1.txt
The last bit of cleanup in MRHelpers (not related to MRInput etc).
Changes
- Add a new class called TezAPIHelpers which contains some methods for conf to
payload, etc
- Helper methods for payloads etc removed from MRHelpers, in favor of the
methods in TezAPIHelpers
- Remvoed doJobClient magic. Replaced the important bit of determining which
API to use with configureMRApiUsage
- Renamed most of the methods in MRHelpers, and improved javadoc to indicate
these are just helpers to parse out existing MR config values.
[~bikassaha], please review.
> 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
>
>
> - 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)
