[ 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)