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

[Francisco De Melo Junior (Jira)]

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

Francisco De Melo Junior commented on JGRP-2130:
------------------------------------------------

I believe this affects UDP, TCP besides NAKACK2 and UNICAST3

> 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)