[jboss-jira] [JBoss JIRA] Commented: (JBCACHE-826) RegionManager region map should be ConcurrentReaderHashMap

[Brian Stansberry (JIRA)]

    [ http://jira.jboss.com/jira/browse/JBCACHE-826?page=comments#action_12355173 ] 
            
Brian Stansberry commented on JBCACHE-826:
------------------------------------------

Same thing needs to be done in 1.4.x for the marshalling region manager. If both marshalling and eviction are enabled, each request will have to do at least two gets on a synchronized map.

> RegionManager region map should be ConcurrentReaderHashMap
> ----------------------------------------------------------
>
>                 Key: JBCACHE-826
>                 URL: http://jira.jboss.com/jira/browse/JBCACHE-826
>             Project: JBoss Cache
>          Issue Type: Task
>      Security Level: Public(Everyone can see) 
>            Reporter: Brian Stansberry
>         Assigned To: Brian Stansberry
>            Priority: Minor
>
> The regionMap in o.j.c.eviction.RegionManager is a Collections.synchronizedMap.  This should be a ConcurrentReaderHashMap.  If eviction is used, for every call the EvictionInterceptor makes a call that ends up doing a lookup in this map, so the synchronization becomes a choke point.
> I already made this change in HEAD along with some refactoring work on eviction configuration.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://jira.jboss.com/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira