[ 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)