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

Daniel Templeton commented on YARN-6112:
----------------------------------------

[~kasha], the issue is that the call to 
{{fsOpDurations.addUpdateCallDuration()}} now (post 4752) happens inside the 
{{if (LOG.isDebugEnabled()}} block.  Before it was done whether debug was 
enabled or not, which appears to be the right thing.

> fsOpDurations.addUpdateCallDuration() should be independent to LOG level
> ------------------------------------------------------------------------
>
>                 Key: YARN-6112
>                 URL: https://issues.apache.org/jira/browse/YARN-6112
>             Project: Hadoop YARN
>          Issue Type: Bug
>          Components: fairscheduler
>            Reporter: Yufei Gu
>            Assignee: Yufei Gu
>         Attachments: YARN-6112.001.patch
>
>
> In the update thread of Fair Scheduler, the 
> {{fsOpDurations.addUpdateCallDuration()}} records the duration of 
> {{update()}}, it should be independent to LOG level. YARN-4752 put the it 
> inside a {{LOG.isDebugEnabled()}} block. Not sure any particular reason to do 
> that. cc [~kasha]



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

---------------------------------------------------------------------
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org

Reply via email to