[jboss-cvs] JBoss Messaging SVN: r5399 - trunk/src/main/org/jboss/messaging/core/postoffice/impl.

[jboss-cvs-commits at lists.jboss.org]

Author: timfox
Date: 2008-11-20 12:14:40 -0500 (Thu, 20 Nov 2008)
New Revision: 5399

Modified:
   trunk/src/main/org/jboss/messaging/core/postoffice/impl/PostOfficeImpl.java
Log:
Added FIXME


Modified: trunk/src/main/org/jboss/messaging/core/postoffice/impl/PostOfficeImpl.java
===================================================================
--- trunk/src/main/org/jboss/messaging/core/postoffice/impl/PostOfficeImpl.java	2008-11-20 11:49:41 UTC (rev 5398)
+++ trunk/src/main/org/jboss/messaging/core/postoffice/impl/PostOfficeImpl.java	2008-11-20 17:14:40 UTC (rev 5399)
@@ -459,6 +459,10 @@
          queues.put(binding.getQueue().getPersistenceID(), binding.getQueue());
       }
       // TODO: This is related to http://www.jboss.com/index.html?module=bb&op=viewtopic&t=145597
+      
+      //FIXME This is incorrect - you cannot assume there is an allowable address in existence
+      //for every address in the post office.
+      //This code is unnecessary if paging stores are loaded lazily
       HashSet<SimpleString> addresses = new HashSet<SimpleString>();
       
       for (Binding binding: bindings)