[ https://issues.apache.org/jira/browse/YARN-5699?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15576895#comment-15576895 ]
Sangjin Lee commented on YARN-5699: ----------------------------------- The latest patch LGTM. I'll wait until tomorrow to give folks time to chime in before I commit. [~rohithsharma], just to be clear, is the intent to commit this to trunk? If I take the latest patch (https://issues.apache.org/jira/secure/attachment/12833383/0003-YARN-5699.patch) it does not apply cleanly to the YARN-5355 branch. Could you please upload the corresponding YARN-5355 patch? > Retrospect yarn entity fields which are publishing in events info fields. > ------------------------------------------------------------------------- > > Key: YARN-5699 > URL: https://issues.apache.org/jira/browse/YARN-5699 > Project: Hadoop YARN > Issue Type: Sub-task > Reporter: Rohith Sharma K S > Assignee: Rohith Sharma K S > Attachments: 0001-YARN-5699.YARN-5355.patch, 0001-YARN-5699.patch, > 0002-YARN-5699.YARN-5355.patch, 0002-YARN-5699.patch, 0003-YARN-5699.patch > > > Currently, all the container information are published at 2 places. Some of > them are at entity info(top-level) and some are at event info. > For containers, some of the event info should be published at container info > level. For example : container exist status, container state, createdTime, > finished time. These are general information to container required for > container-report. So it is better to publish at top level info field. -- This message was sent by Atlassian JIRA (v6.3.4#6332) --------------------------------------------------------------------- To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org