Author: gtully
Date: Mon Jul 20 09:59:03 2009
New Revision: 795733

URL: http://svn.apache.org/viewvc?rev=795733&view=rev
Log:
defer task start till cursor is started - 
https://issues.apache.org/activemq/browse/AMQ-1112

Modified:
    
activemq/trunk/activemq-core/src/main/java/org/apache/activemq/broker/region/Queue.java

Modified: 
activemq/trunk/activemq-core/src/main/java/org/apache/activemq/broker/region/Queue.java
URL: 
http://svn.apache.org/viewvc/activemq/trunk/activemq-core/src/main/java/org/apache/activemq/broker/region/Queue.java?rev=795733&r1=795732&r2=795733&view=diff
==============================================================================
--- 
activemq/trunk/activemq-core/src/main/java/org/apache/activemq/broker/region/Queue.java
 (original)
+++ 
activemq/trunk/activemq-core/src/main/java/org/apache/activemq/broker/region/Queue.java
 Mon Jul 20 09:59:03 2009
@@ -186,10 +186,6 @@
             this.taskRunner = new DeterministicTaskRunner(this.executor,this);
         }
         
-        if (getExpireMessagesPeriod() > 0) {
-            scheduler.schedualPeriodically(expireMessagesTask, 
getExpireMessagesPeriod());
-        }
-        
         super.initialize();
         if (store != null) {
             // Restore the persistent messages.
@@ -631,6 +627,9 @@
         }
         systemUsage.getMemoryUsage().addUsageListener(this);
         messages.start();
+        if (getExpireMessagesPeriod() > 0) {
+            scheduler.schedualPeriodically(expireMessagesTask, 
getExpireMessagesPeriod());
+        }
         doPageIn(false);
     }
 
@@ -642,7 +641,6 @@
             this.executor.shutdownNow();
         }
         
-        LOG.info(toString() + ", canceling expireMessagesTask");
         scheduler.cancel(expireMessagesTask);
         
         if (messages != null) {


Reply via email to