[jboss-jira] [JBoss JIRA] (JGRP-2130) Log: cache result of log.isTraceEnabled()

[Bela Ban (Jira)]

    [ https://issues.redhat.com/browse/JGRP-2130?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14002443#comment-14002443 ] 

Bela Ban commented on JGRP-2130:
--------------------------------

Correct, we do use {{is_trace}} in {{TP}}, which is extended by {{UDP}} and {{TCP}}.

> Log: cache result of log.isTraceEnabled()
> -----------------------------------------
>
>                 Key: JGRP-2130
>                 URL: https://issues.redhat.com/browse/JGRP-2130
>             Project: JGroups
>          Issue Type: Enhancement
>            Reporter: Bela Ban
>            Assignee: Bela Ban
>            Priority: Minor
>              Labels: CR1
>             Fix For: 4.0
>
>
> e.g. 
> {noformat}
> boolean trace=log.isTraceEnabled();
> loop {
>    if(trace) ...
> }
> {noformat}
> Unfortunately, {{isTraceEnabled()}} is not free in most scenarios, so the cost of invoking this method should be amortized over multiple calls.
> I don't want to cache this at a global level, or else we wouldn't be able to change the log level at runtime...



--
This message was sent by Atlassian Jira
(v7.13.8#713008)