[ 
https://issues.apache.org/jira/browse/YARN-4205?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14908204#comment-14908204
 ] 

nijel commented on YARN-4205:
-----------------------------

Thanks [~leftnoteasy] for the comments
Sorry for the small mistakes

Updated the patch
bq. RMAppLifeTimeMonitorService.rmApps -> applicationIdToLifetime? (or shorter 
name if you prefer), it's not rmApps actually
changed to monitoredapps

bq. public synchronized void unregister, synchronized could be removed?
Done

Other comments are fixed.

> Add a service for monitoring application life time out
> ------------------------------------------------------
>
>                 Key: YARN-4205
>                 URL: https://issues.apache.org/jira/browse/YARN-4205
>             Project: Hadoop YARN
>          Issue Type: Sub-task
>          Components: scheduler
>            Reporter: nijel
>            Assignee: nijel
>         Attachments: YARN-4205_01.patch, YARN-4205_02.patch
>
>




--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to