EHC ❯ Race in CacheManager.replaceCacheWithDecoratedCache(Ehcache, Ehcache)

• Type: Bug
• Status: Closed
• Priority: 2 Major
• Resolution: Fixed
• Component/s:
• Labels:

• Assignee: alexsnaps
• Reporter: alexsnaps
• Created: April 09, 2010
• Votes: 0
• Watchers: 0
• Updated:

  July 15, 2010
• Resolved:

  June 16, 2010

Description

There is still a race condition in this method, where the replace isn’t actually checking for the Cache being replaced to still be the same instance. One solution I can think of is replacing the Map<String, Ehcache> ehcaches to have its values replaced by some CacheHolder that would properly override the Object.equals(Object): boolean to check for object identity when the ConcurrentMap.replace is actually being executed

Comments

Alexander Snaps 2010-06-16

Looks like this works actually. equals being based on the cache’s GUID and CacheManager.replaceCacheWithDecoratedCache does the replace. I even wrote a test for that, proving it did work… Probably forgot to close this…