[ 
https://issues.apache.org/jira/browse/SLING-4999?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14947471#comment-14947471
 ] 

Robert Munteanu commented on SLING-4999:
----------------------------------------

Applied in [r1707375|https://svn.apache.org/r1707375]. As I've said, I don't 
have the numbers at hand, and under medium concurrency ( 8 concurrent requests 
) Sling seems to behave more or less the same.

> Check synchronization in RequestProcessorMBeanImpl
> --------------------------------------------------
>
>                 Key: SLING-4999
>                 URL: https://issues.apache.org/jira/browse/SLING-4999
>             Project: Sling
>          Issue Type: Improvement
>          Components: Engine
>    Affects Versions: Engine 2.4.2
>            Reporter: Carsten Ziegeler
>            Assignee: Robert Munteanu
>             Fix For: Engine 2.4.4
>
>         Attachments: 
> 0001-SLING-4999-Check-synchronization-in-RequestProcessor.patch
>
>
> Each and every request goes through RequestProcessorMBeanImpl#addRequestData 
> which is a synchronized block. I think this is a rather heavy bottleneck for 
> request processing.
> In addition, some of the methods which do additional calculation are sync'ed 
> (like getStandardDeviationDurationMsec) but others are not (like 
> getStandardDeviationPeakRecursionDepth). This seems a little bit arbitrary to 
> me



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to