[jboss-cvs] JBoss Messaging SVN: r1738 - branches/Branch_Client_Failover_Experiment/src/main/org/jboss/jms/server/connectionfactory

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

Author: clebert.suconic at jboss.com
Date: 2006-12-09 11:15:23 -0500 (Sat, 09 Dec 2006)
New Revision: 1738

Modified:
   branches/Branch_Client_Failover_Experiment/src/main/org/jboss/jms/server/connectionfactory/ConnectionFactoryJNDIMapper.java
Log:
Just adding a simple comment

Modified: branches/Branch_Client_Failover_Experiment/src/main/org/jboss/jms/server/connectionfactory/ConnectionFactoryJNDIMapper.java
===================================================================
--- branches/Branch_Client_Failover_Experiment/src/main/org/jboss/jms/server/connectionfactory/ConnectionFactoryJNDIMapper.java	2006-12-09 15:22:51 UTC (rev 1737)
+++ branches/Branch_Client_Failover_Experiment/src/main/org/jboss/jms/server/connectionfactory/ConnectionFactoryJNDIMapper.java	2006-12-09 16:15:23 UTC (rev 1738)
@@ -27,11 +27,9 @@
 import java.util.Iterator;
 import java.util.List;
 import java.util.Map;
-
 import javax.naming.Context;
 import javax.naming.InitialContext;
 import javax.naming.NamingException;
-
 import org.jboss.jms.client.JBossConnectionFactory;
 import org.jboss.jms.client.delegate.ClientConnectionFactoryDelegate;
 import org.jboss.jms.client.delegate.ClusteredClientConnectionFactoryDelegate;
@@ -415,10 +413,13 @@
          
          recalculateFailoverMap(nodeAddressMap);
       }
-      
-      //Now we find the "mainDelegate"
-      //TODO - why do we need a "mainDelegate" ????
-      
+
+      // The main delegated is needed for the construction of ClusteredClientConnectionFactoryDelegate
+      // ClusteredClientConnectionFactoryDelegate extends ClientConnectionFactoryDelegate and it will
+      // need the current server's delegate properties to be bound to ObjectId, ServerLocator and
+      // other connection properties.
+      //
+      // The ClusteredCFDelegate will copy these properties on its contructor defined bellow after this loop
       ClientConnectionFactoryDelegate mainDelegate = null;
       
       for(Iterator i = localDelegates.values().iterator(); i.hasNext();)