[ https://issues.apache.org/jira/browse/GEODE-2141?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15713658#comment-15713658 ]
ASF subversion and git services commented on GEODE-2141: -------------------------------------------------------- Commit 50320dc4e828c900b6e1ee92c909fa019cae26d0 in incubator-geode's branch refs/heads/develop from adongre [ https://git-wip-us.apache.org/repos/asf?p=incubator-geode.git;h=50320dc ] GEODE-2141: Replace use of List with ConcurrentHashSet for storing various stats and listener. Moved methods stopNotifier_IfEnabledAndRunning and startNotifier_IfEnabledAndNotRunning in synchronized block. GEODE-2141: Addresing the review comments. GEODE-2141: Addressing Review Comments Making addMonitor and remoteMonitor threadsafe. > StatisticsMonitor and StatMonitorHandler should use ConcurrentHashSet to > store monitors, listeners and statisticIds > ------------------------------------------------------------------------------------------------------------------- > > Key: GEODE-2141 > URL: https://issues.apache.org/jira/browse/GEODE-2141 > Project: Geode > Issue Type: Improvement > Reporter: Avinash Dongre > Assignee: Avinash Dongre > > In StatisticsMonitor and StatMonitorHandler List is used to store monitors, > listeners and statisticIds. > We should be using ConcurrentHashSet for performance Reason. > In my local testing When I replace the List to > com.gemstone.gemfire.internal.concurrent. > ConcurrentHashSet > I see significant improvement in creating large number of region creation. > ( from ~7hrs to ~26 minutes) > More details about the same is on dev list. > Refer : http://markmail.org/message/o7td3fczylx4uaet -- This message was sent by Atlassian JIRA (v6.3.4#6332)