[jboss-cvs] JBoss Messaging SVN: r1749 - branches/Branch_Client_Failover_Experiment/src/main/org/jboss/jms/client/container

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

Author: clebert.suconic at jboss.com
Date: 2006-12-09 15:45:46 -0500 (Sat, 09 Dec 2006)
New Revision: 1749

Modified:
   branches/Branch_Client_Failover_Experiment/src/main/org/jboss/jms/client/container/HAAspect.java
Log:
just adding a TODO

Modified: branches/Branch_Client_Failover_Experiment/src/main/org/jboss/jms/client/container/HAAspect.java
===================================================================
--- branches/Branch_Client_Failover_Experiment/src/main/org/jboss/jms/client/container/HAAspect.java	2006-12-09 20:26:32 UTC (rev 1748)
+++ branches/Branch_Client_Failover_Experiment/src/main/org/jboss/jms/client/container/HAAspect.java	2006-12-09 20:45:46 UTC (rev 1749)
@@ -26,9 +26,7 @@
 import java.util.Iterator;
 import java.util.List;
 import java.util.Map;
-
 import javax.jms.JMSException;
-
 import org.jboss.aop.joinpoint.Invocation;
 import org.jboss.aop.joinpoint.MethodInvocation;
 import org.jboss.aop.metadata.SimpleMetaData;
@@ -83,7 +81,9 @@
    public Object handleCreateConnectionDelegate(Invocation invocation) throws Throwable
    {
       if (getServers(invocation) != null)
-      {         
+      {
+         // TODO: this should be in loop while we get exceptions creating connections, always trying
+         //       the next Delegate when we get an exception.
          log.info("Clustered createConnection");
                   
          //In a clustered configuration we create connections in a round-robin fashion