[ 
http://jira.qos.ch/browse/LBCLASSIC-153?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Ceki Gulcu closed LBCLASSIC-153.
--------------------------------

    Resolution: Cannot Reproduce

Hello,

I have tried to reproduce this bug by creating a relevant test case, called 
ReconfigurePerf, part of the  ch.qos.logback.classic.turbo package (in 
src/test) and measuring its performance on 4 core CPU with 5 threads. JProfiler 
showed that only a marginal percentage of CPU was spent in 
ReconfigureOnChangeFilter's decide method. Using Yourkit yielded similar 
results. 

So, either

1) ReconfigurePerf is non-representative
2) I misused the profiler
3) your tests are non-representative

In the mean-time I am marking this bug as "Cannot reproduce". If after 
profiling ReconfigurePerf you obtain a different result, or you detect an error 
in the test, do not hesitate to reopen this report.

> Excessive synchronization in ReconfigureOnChangeFilter.decide
> -------------------------------------------------------------
>
>                 Key: LBCLASSIC-153
>                 URL: http://jira.qos.ch/browse/LBCLASSIC-153
>             Project: logback-classic
>          Issue Type: Bug
>    Affects Versions: 0.917
>            Reporter: Anders Wallgren
>            Assignee: Ceki Gulcu
>         Attachments: screenshot-1.jpg
>
>
> The synchronization in ReconfigureOnChangeFilter.decide hurts scalability.
> It seems unnecessary to serialize the code in changeDetected -- it should be 
> possible to use atomic updates of nextCheck and lastModified and only 
> serialize the actual reconfiguration.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: 
http://jira.qos.ch/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        
_______________________________________________
logback-dev mailing list
logback-dev@qos.ch
http://qos.ch/mailman/listinfo/logback-dev

Reply via email to