logging-log4j-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From bugzi...@apache.org
Subject DO NOT REPLY [Bug 51047] Move org.apache.log4j.Category to reentrant read/write locks
Date Mon, 11 Apr 2011 14:18:19 GMT

--- Comment #4 from Bartek <kowalewski.bartosz@gmail.com> 2011-04-11 10:18:16 EDT ---
We observed this issue when doing performance tests with 80-160 tps (and plenty
of threads running in parallel). The app was later profiled and lots of threads
were waiting on this synchronization. Changing these mechanism helped as the
only synchronization that is now on the path is in the appender. As we have
more than one appender, log4j now handles the number of threads that we use
better - improved thread parallelism is observed :-).

AppenderSkeleton doAppend() is synchronized. Removing the old synchronization
mechanism from Category will not cause appenders to malfunction no matter how
concrete appenders are implemented. That's what source code suggests and that's
also what we observed in our tests :).

I also logged an improvement related to synchronization in AsyncAppender:

Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

To unsubscribe, e-mail: log4j-dev-unsubscribe@logging.apache.org
For additional commands, e-mail: log4j-dev-help@logging.apache.org

View raw message