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

Rajesh Balamohan commented on TEZ-2825:
---------------------------------------

+1. very minor comment

inTerminalState() check could be moved into getCompletedTaskProgress() to avoid 
readLock.lock() (as getCompletedTaskProgress already holds the lock)

> Report progress in terms of completed tasks to reduce load on AM for Tez UI
> ---------------------------------------------------------------------------
>
>                 Key: TEZ-2825
>                 URL: https://issues.apache.org/jira/browse/TEZ-2825
>             Project: Apache Tez
>          Issue Type: Bug
>            Reporter: Hitesh Shah
>            Assignee: Hitesh Shah
>         Attachments: TEZ-2825.1.patch, TEZ-2825.2.patch
>
>
> Scanning all tasks and doing an average progress is very heavyweight for 
> calculating vertex progress. 



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

Reply via email to