You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@qpid.apache.org by rg...@apache.org on 2012/02/01 12:57:25 UTC

svn commit: r1239095 - /qpid/trunk/qpid/java/broker/src/main/java/org/apache/qpid/server/queue/SimpleQueueEntryList.java

Author: rgodfrey
Date: Wed Feb  1 11:57:24 2012
New Revision: 1239095

URL: http://svn.apache.org/viewvc?rev=1239095&view=rev
Log:
QPID-3780 : guard against the possibility that the HWM is older than the queue head

Modified:
    qpid/trunk/qpid/java/broker/src/main/java/org/apache/qpid/server/queue/SimpleQueueEntryList.java

Modified: qpid/trunk/qpid/java/broker/src/main/java/org/apache/qpid/server/queue/SimpleQueueEntryList.java
URL: http://svn.apache.org/viewvc/qpid/trunk/qpid/java/broker/src/main/java/org/apache/qpid/server/queue/SimpleQueueEntryList.java?rev=1239095&r1=1239094&r2=1239095&view=diff
==============================================================================
--- qpid/trunk/qpid/java/broker/src/main/java/org/apache/qpid/server/queue/SimpleQueueEntryList.java (original)
+++ qpid/trunk/qpid/java/broker/src/main/java/org/apache/qpid/server/queue/SimpleQueueEntryList.java Wed Feb  1 11:57:24 2012
@@ -193,7 +193,7 @@ public class SimpleQueueEntryList implem
         else
         {
             SimpleQueueEntryImpl unscavengedHWM = _unscavengedHWM.get();
-            if(unscavengedHWM != null && unscavengedHWM.compareTo(next) < 0)
+            if(unscavengedHWM != null && (next == null || unscavengedHWM.compareTo(next) < 0))
             {
                 _unscavengedHWM.compareAndSet(unscavengedHWM, null);
             }



---------------------------------------------------------------------
Apache Qpid - AMQP Messaging Implementation
Project:      http://qpid.apache.org
Use/Interact: mailto:commits-subscribe@qpid.apache.org