[ https://issues.apache.org/jira/browse/YARN-3044?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14380622#comment-14380622 ]
Naganarasimha G R commented on YARN-3044: ----------------------------------------- Thanks [~vrushalic], for reviewing it bq. moving the code across packages may break software which is importing these. Well the classes which are being moved are very much private to the System metrics publisher(event classes ) and are not exposed outside. So as the same set of events will be used by RM timeline collector i have moved it out, which i feel should be fine. may be [~zjshen] and [~sjlee0] can also opine on the same. bq. I think we should be supporting all states as given in org.apache.hadoop.yarn.api.records.YarnApplicationState, no? Well i too had the same doubt whether we need to capture very minute part of the life cycle events as present in the RMAppState(/YarnApplicationState) and RMAppAttemptState. That is the reason i had posted earlier but [~sjlee0] had opined to start of with the basic ones hence provided this patch. Based on the input from all will add other required states > [Event producers] Implement RM writing app lifecycle events to ATS > ------------------------------------------------------------------ > > Key: YARN-3044 > URL: https://issues.apache.org/jira/browse/YARN-3044 > Project: Hadoop YARN > Issue Type: Sub-task > Components: timelineserver > Reporter: Sangjin Lee > Assignee: Naganarasimha G R > Attachments: YARN-3044.20150325-1.patch > > > Per design in YARN-2928, implement RM writing app lifecycle events to ATS. -- This message was sent by Atlassian JIRA (v6.3.4#6332)