You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@activemq.apache.org by gt...@apache.org on 2012/06/05 16:48:18 UTC

svn commit: r1346401 - /activemq/trunk/activemq-core/src/main/java/org/apache/activemq/store/jdbc/adapter/DefaultJDBCAdapter.java

Author: gtully
Date: Tue Jun  5 14:48:17 2012
New Revision: 1346401

URL: http://svn.apache.org/viewvc?rev=1346401&view=rev
Log:
https://issues.apache.org/jira/browse/AMQ-3288 - fix over flow to negative on signed int accumulator, would break priority message deletion

Modified:
    activemq/trunk/activemq-core/src/main/java/org/apache/activemq/store/jdbc/adapter/DefaultJDBCAdapter.java

Modified: activemq/trunk/activemq-core/src/main/java/org/apache/activemq/store/jdbc/adapter/DefaultJDBCAdapter.java
URL: http://svn.apache.org/viewvc/activemq/trunk/activemq-core/src/main/java/org/apache/activemq/store/jdbc/adapter/DefaultJDBCAdapter.java?rev=1346401&r1=1346400&r2=1346401&view=diff
==============================================================================
--- activemq/trunk/activemq-core/src/main/java/org/apache/activemq/store/jdbc/adapter/DefaultJDBCAdapter.java (original)
+++ activemq/trunk/activemq-core/src/main/java/org/apache/activemq/store/jdbc/adapter/DefaultJDBCAdapter.java Tue Jun  5 14:48:17 2012
@@ -805,7 +805,7 @@ public class DefaultJDBCAdapter implemen
         }
     }
 
-    int priorityIterator = 0;
+    char priorityIterator = 0; // unsigned
     public void doDeleteOldMessages(TransactionContext c) throws SQLException, IOException {
         PreparedStatement s = null;
         cleanupExclusiveLock.writeLock().lock();