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

jboss-cvs-commits at lists.jboss.org jboss-cvs-commits at lists.jboss.org
Sat Jun 20 16:34:33 EDT 2009


Author: clebert.suconic at jboss.com
Date: 2009-06-20 16:34:33 -0400 (Sat, 20 Jun 2009)
New Revision: 7413

Modified:
   trunk/src/main/org/jboss/messaging/core/server/impl/ServerConsumerImpl.java
Log:
Just simplifying a comment

Modified: trunk/src/main/org/jboss/messaging/core/server/impl/ServerConsumerImpl.java
===================================================================
--- trunk/src/main/org/jboss/messaging/core/server/impl/ServerConsumerImpl.java	2009-06-20 16:39:19 UTC (rev 7412)
+++ trunk/src/main/org/jboss/messaging/core/server/impl/ServerConsumerImpl.java	2009-06-20 20:34:33 UTC (rev 7413)
@@ -535,11 +535,8 @@
    {
       if (messageQueue.isBackup())
       {
-         // We are supposed to finish largeMessageDeliverer, or use all the possible credits before we return this
-         // method.
-         // If we play the commands on a different order than how they were generated on the live node, we will
-         // eventually still be running this largeMessage before the next message come, what would reject messages
-         // from the cluster
+         // We need to deliver the largeMessage on backup also, exactly as done on the live node.
+         // In case of failure the same packets will be available for resume sending.
          largeMessageDeliverer.deliver();
       }
       else




More information about the jboss-cvs-commits mailing list