You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@activemq.apache.org by gt...@apache.org on 2011/01/26 13:14:08 UTC
svn commit: r1063691 -
/activemq/trunk/kahadb/src/main/java/org/apache/kahadb/journal/Journal.java
Author: gtully
Date: Wed Jan 26 12:14:07 2011
New Revision: 1063691
URL: http://svn.apache.org/viewvc?rev=1063691&view=rev
Log:
https://issues.apache.org/jira/browse/AMQ-3160 - ConcurrentModificationException in ActiveMQ Journal Checkpoint Worker
access to the file map needs to have a sync so that it can get an accurate snapshot and avoid concurrent access ex
Modified:
activemq/trunk/kahadb/src/main/java/org/apache/kahadb/journal/Journal.java
Modified: activemq/trunk/kahadb/src/main/java/org/apache/kahadb/journal/Journal.java
URL: http://svn.apache.org/viewvc/activemq/trunk/kahadb/src/main/java/org/apache/kahadb/journal/Journal.java?rev=1063691&r1=1063690&r2=1063691&view=diff
==============================================================================
--- activemq/trunk/kahadb/src/main/java/org/apache/kahadb/journal/Journal.java (original)
+++ activemq/trunk/kahadb/src/main/java/org/apache/kahadb/journal/Journal.java Wed Jan 26 12:14:07 2011
@@ -684,7 +684,7 @@ public class Journal {
return fileByFileMap.keySet();
}
- public Map<Integer, DataFile> getFileMap() {
+ public synchronized Map<Integer, DataFile> getFileMap() {
return new TreeMap<Integer, DataFile>(fileMap);
}