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

Prasanth Jayachandran commented on HIVE-20772:
----------------------------------------------

tezCounters.findCounter(LlapExecutorCounters.EXECUTOR_CPU_NS).increment(cpuTime);

Should this be set instead of increment? If we are reusing the executor thread 
(I hope we are not), increment will give aggregate whereas set will just give 
just for that thread. 

 

Looks good otherwise. 

> record per-task CPU counters in LLAP
> ------------------------------------
>
>                 Key: HIVE-20772
>                 URL: https://issues.apache.org/jira/browse/HIVE-20772
>             Project: Hive
>          Issue Type: Bug
>            Reporter: Sergey Shelukhin
>            Assignee: Sergey Shelukhin
>            Priority: Major
>         Attachments: HIVE-20772.patch
>
>




--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to