[OAK-5521] CommunicationObserver and its MBeans need proper synchronization

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.7.11, 1.8.0
• Component/s: segment-tar
• Assignee: Francesco Mari
• Reporter: Francesco Mari
• Created: 26/Jan/17 10:01
• Updated: 13/Nov/17 13:55
• Resolved: 26/Oct/17 13:40

Description

The CommunicationObserver used in Cold Standby might be called concurrently by multiple threads. The implementation of CommunicationObserver doesn't currently use any form of synchronization to protect its state.

Moreover, the MBeans published by the CommunicationObserver might be concurrently read via JMX and updated via the CommunicationObserver. As such, further synchronization is also needed in the MBeans to protect concurrent access to shared data.