You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@tez.apache.org by "Devaraj K (JIRA)" <ji...@apache.org> on 2014/04/29 10:41:14 UTC

[jira] [Commented] (TEZ-1090) Micro optimization - Remove duplicate updateProcessTree() in TaskCounterUpdater

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

Devaraj K commented on TEZ-1090:
--------------------------------

Code change looks good, can you also remove this unused variable as part of this patch.

{code:xml}
  private long initialCpuCumulativeTime;
{code}


> Micro optimization - Remove duplicate updateProcessTree() in TaskCounterUpdater
> -------------------------------------------------------------------------------
>
>                 Key: TEZ-1090
>                 URL: https://issues.apache.org/jira/browse/TEZ-1090
>             Project: Apache Tez
>          Issue Type: Improvement
>            Reporter: Rajesh Balamohan
>            Assignee: Rajesh Balamohan
>             Fix For: 0.5.0
>
>         Attachments: TEZ-1090-v1.patch
>
>
> In TaskCounterUpdater, updateProcessTree() is invoked from initResourceCalculatorPlugin & recordInitialCpuStats.  With ResourceCalculatorProcessTree, close to 300-500 ms is wasted in every task launch.



--
This message was sent by Atlassian JIRA
(v6.2#6252)