]
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...