Re: [infinispan-dev] if (trace) logger.tracef - it makes sense

2016-10-01 Thread Dan Berindei
On Fri, Sep 30, 2016 at 10:15 PM, David M. Lloyd wrote: > The performance problem that this trick is meant to resolve is really a > problem in the logging backend. It *should* be faster inside of > WildFly, where JBoss LogManager is used, because that project just > checks a single volatile field

Re: [infinispan-dev] if (trace) logger.tracef - it makes sense

2016-10-01 Thread Dan Berindei
Dennis, this has been done in Infinispan ever since 4.0, so feel free to open a BZ :) Cheers Dan On Fri, Sep 30, 2016 at 7:16 PM, Dennis Reed wrote: > As Wolf noted, caching the trace flag is bad when trying to debug issues. > > Don't do it! It's not worth breaking the logging semantics for a