You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@activemq.apache.org by ch...@apache.org on 2008/02/22 18:03:43 UTC
svn commit: r630253 -
/activemq/trunk/activemq-core/src/main/java/org/apache/activemq/store/amq/AMQPersistenceAdapter.java
Author: chirino
Date: Fri Feb 22 09:03:39 2008
New Revision: 630253
URL: http://svn.apache.org/viewvc?rev=630253&view=rev
Log:
https://issues.apache.org/activemq/browse/AMQ-1595
Modified:
activemq/trunk/activemq-core/src/main/java/org/apache/activemq/store/amq/AMQPersistenceAdapter.java
Modified: activemq/trunk/activemq-core/src/main/java/org/apache/activemq/store/amq/AMQPersistenceAdapter.java
URL: http://svn.apache.org/viewvc/activemq/trunk/activemq-core/src/main/java/org/apache/activemq/store/amq/AMQPersistenceAdapter.java?rev=630253&r1=630252&r2=630253&view=diff
==============================================================================
--- activemq/trunk/activemq-core/src/main/java/org/apache/activemq/store/amq/AMQPersistenceAdapter.java (original)
+++ activemq/trunk/activemq-core/src/main/java/org/apache/activemq/store/amq/AMQPersistenceAdapter.java Fri Feb 22 09:03:39 2008
@@ -85,7 +85,7 @@
private static final Log LOG = LogFactory.getLog(AMQPersistenceAdapter.class);
private final ConcurrentHashMap<ActiveMQQueue, AMQMessageStore> queues = new ConcurrentHashMap<ActiveMQQueue, AMQMessageStore>();
- private final ConcurrentHashMap<ActiveMQTopic, AMQMessageStore> topics = new ConcurrentHashMap<ActiveMQTopic, AMQMessageStore>();
+ private final ConcurrentHashMap<ActiveMQTopic, AMQTopicMessageStore> topics = new ConcurrentHashMap<ActiveMQTopic, AMQTopicMessageStore>();
private static final String PROPERTY_PREFIX = "org.apache.activemq.store.amq";
private static final boolean BROKEN_FILE_LOCK;
private static final boolean DISABLE_LOCKING;
@@ -266,14 +266,14 @@
Scheduler.cancel(periodicCheckpointTask);
Scheduler.cancel(periodicCleanupTask);
}
- Iterator<AMQMessageStore> iterator = queues.values().iterator();
- while (iterator.hasNext()) {
- AMQMessageStore ms = iterator.next();
+ Iterator<AMQMessageStore> queueIterator = queues.values().iterator();
+ while (queueIterator.hasNext()) {
+ AMQMessageStore ms = queueIterator.next();
ms.stop();
}
- iterator = topics.values().iterator();
- while (iterator.hasNext()) {
- final AMQTopicMessageStore ms = (AMQTopicMessageStore)iterator.next();
+ Iterator<AMQTopicMessageStore> topicIterator = topics.values().iterator();
+ while (topicIterator.hasNext()) {
+ final AMQTopicMessageStore ms = topicIterator.next();
ms.stop();
}
// Take one final checkpoint and stop checkpoint processing.
@@ -344,17 +344,17 @@
}
Location newMark = null;
- Iterator<AMQMessageStore> iterator = queues.values().iterator();
- while (iterator.hasNext()) {
- final AMQMessageStore ms = iterator.next();
+ Iterator<AMQMessageStore> queueIterator = queues.values().iterator();
+ while (queueIterator.hasNext()) {
+ final AMQMessageStore ms = queueIterator.next();
Location mark = (Location)ms.getMark();
if (mark != null && (newMark == null || newMark.compareTo(mark) < 0)) {
newMark = mark;
}
}
- iterator = topics.values().iterator();
- while (iterator.hasNext()) {
- final AMQTopicMessageStore ms = (AMQTopicMessageStore)iterator.next();
+ Iterator<AMQTopicMessageStore> topicIterator = topics.values().iterator();
+ while (topicIterator.hasNext()) {
+ final AMQTopicMessageStore ms = topicIterator.next();
Location mark = (Location)ms.getMark();
if (mark != null && (newMark == null || newMark.compareTo(mark) < 0)) {
newMark = mark;
@@ -430,7 +430,7 @@
}
public TopicMessageStore createTopicMessageStore(ActiveMQTopic destinationName) throws IOException {
- AMQTopicMessageStore store = (AMQTopicMessageStore)topics.get(destinationName.getPhysicalName());
+ AMQTopicMessageStore store = topics.get(destinationName);
if (store == null) {
TopicReferenceStore checkpointStore = referenceStoreAdapter.createTopicReferenceStore(destinationName);
store = new AMQTopicMessageStore(this,checkpointStore, destinationName);