Issue Details (XML | Word | Printable)

Key: JRA-12600
Type: Bug Bug
Status: Resolved Resolved
Resolution: Fixed
Priority: Major Major
Assignee: Jed Wesley-Smith [Atlassian]
Reporter: Nick Menere [Atlassian]
Votes: 0
Watchers: 0
Operations

If you were logged in you would be able to see more operations.
JIRA

CachedGenericConfigManager is not thread safe

Created: 19/Apr/07 08:23 PM   Updated: 09/Jan/08 09:22 PM
Component/s: Backend / Domain Model
Affects Version/s: None
Fix Version/s: 3.10.2

Time Tracking:
Original Estimate: 6 hours
Original Estimate - 6 hours
Remaining Estimate: 6 hours
Remaining Estimate - 6 hours
Time Spent: Not Specified
Remaining Estimate - 6 hours

Issue Links:
Duplicate
 

Participants: Dylan Etkin [Atlassian], Jed Wesley-Smith [Atlassian] and Nick Menere [Atlassian]
Since last comment: 1 year, 5 weeks, 3 days ago
Resolution Date: 30/Aug/07 09:51 PM
To be done by: Single developer
Labels:


 Description  « Hide
The CachedGenericConfigManager contains a non thread safe map.

It contains the code:

public void refreshCache()
    {
        cache = MultiKeyMap.decorate(new LRUMap(250));
        Collections.synchronizedMap(cache);
    }

But should be:

public void refreshCache()
    {
        cache = MultiKeyMap.decorate(new LRUMap(250));
        cache = Collections.synchronizedMap(cache);
    }

There could be a few more little issues as well, such as when refreshCache is called at the same time as retrieve...



 All   Comments   Work Log   Change History      Sort Order: Ascending order - Click to sort in descending order
Nick Menere [Atlassian] added a comment - 19/Apr/07 08:26 PM
Upgrading to a later version of common-collection may work as well.

Dylan Etkin [Atlassian] added a comment - 30/Aug/07 09:51 PM
This was fixed back on 5/25/07 the issue was never closed.