From 230255a47902e94a2a8cf20f2962615eb3630322 Mon Sep 17 00:00:00 2001
From: "Hiram R. Chirino" <chirino@apache.org>
Date: Fri, 22 Feb 2008 17:03:39 +0000
Subject: [PATCH] https://issues.apache.org/activemq/browse/AMQ-1595

git-svn-id: https://svn.apache.org/repos/asf/activemq/trunk@630253 13f79535-47bb-0310-9956-ffa450edef68
---
 .../store/amq/AMQPersistenceAdapter.java      | 28 +++++++++----------
 1 file changed, 14 insertions(+), 14 deletions(-)

diff --git a/activemq-core/src/main/java/org/apache/activemq/store/amq/AMQPersistenceAdapter.java b/activemq-core/src/main/java/org/apache/activemq/store/amq/AMQPersistenceAdapter.java
index 04a9a2801f..e00e8dc220 100644
--- a/activemq-core/src/main/java/org/apache/activemq/store/amq/AMQPersistenceAdapter.java
+++ b/activemq-core/src/main/java/org/apache/activemq/store/amq/AMQPersistenceAdapter.java
@@ -85,7 +85,7 @@ public class AMQPersistenceAdapter implements PersistenceAdapter, UsageListener,
 
     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 @@ public class AMQPersistenceAdapter implements PersistenceAdapter, UsageListener,
             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 @@ public class AMQPersistenceAdapter implements PersistenceAdapter, UsageListener,
             }
 
             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 class AMQPersistenceAdapter implements PersistenceAdapter, UsageListener,
     }
 
     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);