[
https://issues.apache.org/jira/browse/YARN-4129?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14969858#comment-14969858
]
Sangjin Lee commented on YARN-4129:
-----------------------------------
Thanks for updating the patch [~Naganarasimha]. I'm +1 on the latest patch
(v.4). I'll let others comment on this for today before I commit it. Thanks!
> Refactor the SystemMetricPublisher in RM to better support newer events
> -----------------------------------------------------------------------
>
> Key: YARN-4129
> URL: https://issues.apache.org/jira/browse/YARN-4129
> Project: Hadoop YARN
> Issue Type: Sub-task
> Components: resourcemanager
> Reporter: Naganarasimha G R
> Assignee: Naganarasimha G R
> Attachments: YARN-4129-YARN-2928.002.patch,
> YARN-4129-YARN-2928.003.patch, YARN-4129-YARN-2928.004.patch,
> YARN-4129.YARN-2928.001.patch
>
>
> Currently to add new timeline event/ entity in RM side, one has to add a
> method in publisher and a method in handler and create a new event class
> which looks cumbersome and redundant. also further all the events might not
> be required to be published in V1 & V2. So adopting the approach similar to
> what was adopted in YARN-3045(NM side)
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
