[
https://issues.apache.org/jira/browse/YARN-1931?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13967319#comment-13967319
]
Vinod Kumar Vavilapalli commented on YARN-1931:
-----------------------------------------------
Please don't do this. Two wrongs make a right. These are private APIs. Just
like Records any many other things that MR is still using directly. We should
fix MR to not depend on those, not make methods public which are never intended
to be public.
Also if MR code is the best place to learn how to write apps, this community
has got it all wrong. How about helping with some documentation?
The class itself is marked private. So it won't appear in javadoc. It will
greatly help if developers of ecosystem projects refer to published javadocs
instead of relying on source code.
> Private API change in YARN-1824 in 2.4 broke compatibility with previous
> releases
> ---------------------------------------------------------------------------------
>
> Key: YARN-1931
> URL: https://issues.apache.org/jira/browse/YARN-1931
> Project: Hadoop YARN
> Issue Type: Bug
> Components: applications
> Affects Versions: 2.4.0
> Reporter: Thomas Graves
> Assignee: Sandy Ryza
> Priority: Blocker
> Attachments: YARN-1931-1.patch, YARN-1931.patch
>
>
> YARN-1824 broke compatibility with previous 2.x releases by changes the API's
> in org.apache.hadoop.yarn.util.Apps.{setEnvFromInputString,addToEnvironment}
> The old api should be added back in.
> This affects any ApplicationMasters who were using this api. It also breaks
> previously built MapReduce libraries from working with the new Yarn release
> as MR uses this api.
--
This message was sent by Atlassian JIRA
(v6.2#6252)
