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

Tao Yang commented on YARN-9795:
--------------------------------

Thanks [~fengnanli] for this improvement.
Patch almost LGTM,  IMO, there's no need to set -1 as the initial value of 
scheduledTime and add the special annotation, 0 should be the proper initial 
value like other times.  And new check-style warnings should be fixed as well.

> ClusterMetrics to include AM allocation delay
> ---------------------------------------------
>
>                 Key: YARN-9795
>                 URL: https://issues.apache.org/jira/browse/YARN-9795
>             Project: Hadoop YARN
>          Issue Type: Improvement
>            Reporter: Fengnan Li
>            Assignee: Fengnan Li
>            Priority: Minor
>         Attachments: YARN-9795.001.patch
>
>
> Add AM container allocation in QueueMetrics to help diagnose performance 
> issue. This is following 
> [YARN-2802|https://jira.apache.org/jira/browse/YARN-2802]
>  



--
This message was sent by Atlassian Jira
(v8.3.2#803003)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to