[
https://issues.apache.org/jira/browse/YARN-3543?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14585431#comment-14585431
]
Rohith commented on YARN-3543:
------------------------------
Thanks [~xgong] for the review..
bq. Could we not directly change the ApplicationReport.newInstance() ? This
will break other applications, such as Tez.
IIUC, ApplicationReport#newInstance() is @private annotated, so ohter client
should not able to use this. And in the ealier patch I was added new method
which does not break compatibility, but [~vinodkv] suggested me not to change
this API in his reveiw comment
[link|https://issues.apache.org/jira/browse/YARN-3543?focusedCommentId=14533819&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-14533819]
> ApplicationReport should be able to tell whether the Application is AM
> managed or not.
> ---------------------------------------------------------------------------------------
>
> Key: YARN-3543
> URL: https://issues.apache.org/jira/browse/YARN-3543
> Project: Hadoop YARN
> Issue Type: Improvement
> Components: api
> Affects Versions: 2.6.0
> Reporter: Spandan Dutta
> Assignee: Rohith
> Attachments: 0001-YARN-3543.patch, 0001-YARN-3543.patch,
> 0002-YARN-3543.patch, 0002-YARN-3543.patch, 0003-YARN-3543.patch,
> 0004-YARN-3543.patch, 0004-YARN-3543.patch, 0004-YARN-3543.patch,
> YARN-3543-AH.PNG, YARN-3543-RM.PNG
>
>
> Currently we can know whether the application submitted by the user is AM
> managed from the applicationSubmissionContext. This can be only done at the
> time when the user submits the job. We should have access to this info from
> the ApplicationReport as well so that we can check whether an app is AM
> managed or not anytime.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
