[SLING-4999] Check synchronization in RequestProcessorMBeanImpl

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: Engine 2.4.2
• Fix Version/s: Engine 2.4.4
• Component/s: Engine
• Assignee: Robert Munteanu
• Reporter: Carsten Ziegeler
• Created: 04/Sep/15 07:35
• Updated: 12/Oct/15 09:44
• Resolved: 07/Oct/15 19:52

Description

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