[jboss-cvs] JBoss Messaging SVN: r3236 - in trunk: tests/src/org/jboss/test/messaging/jms/clustering and 1 other directory.

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

Author: timfox
Date: 2007-10-21 10:10:18 -0400 (Sun, 21 Oct 2007)
New Revision: 3236

Modified:
   trunk/src/main/org/jboss/jms/server/endpoint/ServerSessionEndpoint.java
   trunk/tests/src/org/jboss/test/messaging/jms/clustering/MultipleFailoverTest.java
Log:
A couple of tweaks


Modified: trunk/src/main/org/jboss/jms/server/endpoint/ServerSessionEndpoint.java
===================================================================
--- trunk/src/main/org/jboss/jms/server/endpoint/ServerSessionEndpoint.java	2007-10-21 13:46:15 UTC (rev 3235)
+++ trunk/src/main/org/jboss/jms/server/endpoint/ServerSessionEndpoint.java	2007-10-21 14:10:18 UTC (rev 3236)
@@ -1749,9 +1749,6 @@
       if (rec == null)
       {
          log.debug("Cannot find " + ack + " to acknowledge, it was probably acknowledged before");
-         
-         log.info("**** CANNOT FIND ACK TO ACKNOWLEDGED");
-         
          return;
       }
       

Modified: trunk/tests/src/org/jboss/test/messaging/jms/clustering/MultipleFailoverTest.java
===================================================================
--- trunk/tests/src/org/jboss/test/messaging/jms/clustering/MultipleFailoverTest.java	2007-10-21 13:46:15 UTC (rev 3235)
+++ trunk/tests/src/org/jboss/test/messaging/jms/clustering/MultipleFailoverTest.java	2007-10-21 14:10:18 UTC (rev 3236)
@@ -260,6 +260,7 @@
          
          //We check that we received all the message
          //we allow for duplicates, see http://jira.jboss.org/jira/browse/JBMESSAGING-604
+         //We are using auto_ack with a listener so duplicates are ok
          
          if (!list.waitFor(count - 1))
          {