[GitHub] [geode] dschneider-pivotal commented on pull request #2250: GEODE-5314: MBeanStatsMonitor child classes should use atomics instead of volatiles to avoid data race

[GitHub]

You should make clear that it is safe to have multiple concurrent readers of 
this class.
It is only the methods that modify (handleNotification, decreasePrevValues, and 
increaseStats) that need external synchronization.

[ Full content available at: https://github.com/apache/geode/pull/2250 ]
This message was relayed via gitbox.apache.org for devnull@infra.apache.org