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

Wangda Tan commented on YARN-2868:
----------------------------------

[~rchiang],
Just reviewed patch, I'm not sure if you misunderstood what I and Karthik 
meant. I agree with what you mentioned in 
https://issues.apache.org/jira/browse/YARN-2868?focusedCommentId=14274308&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-14274308
 (comment#1) and also Karthik's comment: 
https://issues.apache.org/jira/browse/YARN-2868?focusedCommentId=14274317&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-14274317.
 It's better to keep AtomicLong as what you originally done. Lock application 
from caller is not clear to me.

Thanks,

> Add metric for initial container launch time
> --------------------------------------------
>
>                 Key: YARN-2868
>                 URL: https://issues.apache.org/jira/browse/YARN-2868
>             Project: Hadoop YARN
>          Issue Type: Improvement
>            Reporter: Ray Chiang
>            Assignee: Ray Chiang
>              Labels: metrics, supportability
>         Attachments: YARN-2868-01.patch, YARN-2868.002.patch, 
> YARN-2868.003.patch, YARN-2868.004.patch, YARN-2868.005.patch, 
> YARN-2868.006.patch
>
>
> Add a metric to measure the latency between "starting container allocation" 
> and "first container actually allocated".



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

Reply via email to