activemq-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From tab...@apache.org
Subject svn commit: r1174951 - /activemq/trunk/activemq-core/src/main/java/org/apache/activemq/thread/Scheduler.java
Date Fri, 23 Sep 2011 18:48:19 GMT
Author: tabish
Date: Fri Sep 23 18:48:19 2011
New Revision: 1174951

URL: http://svn.apache.org/viewvc?rev=1174951&view=rev
Log:
fix for: https://issues.apache.org/jira/browse/AMQ-3031	

Don't use the scheduleAtFixedRate method in our scheduler as we
don't really have a need for real time task execution, just use
the fixed delay scheduler so that jobs don't stack up.

Modified:
    activemq/trunk/activemq-core/src/main/java/org/apache/activemq/thread/Scheduler.java

Modified: activemq/trunk/activemq-core/src/main/java/org/apache/activemq/thread/Scheduler.java
URL: http://svn.apache.org/viewvc/activemq/trunk/activemq-core/src/main/java/org/apache/activemq/thread/Scheduler.java?rev=1174951&r1=1174950&r2=1174951&view=diff
==============================================================================
--- activemq/trunk/activemq-core/src/main/java/org/apache/activemq/thread/Scheduler.java (original)
+++ activemq/trunk/activemq-core/src/main/java/org/apache/activemq/thread/Scheduler.java Fri
Sep 23 18:48:19 2011
@@ -23,25 +23,25 @@ import org.apache.activemq.util.ServiceS
 import org.apache.activemq.util.ServiceSupport;
 
 /**
- * 
+ *
  */
-public final class Scheduler extends ServiceSupport { 
+public final class Scheduler extends ServiceSupport {
     private final String name;
-	private Timer timer;
+    private Timer timer;
     private final HashMap<Runnable, TimerTask> timerTasks = new HashMap<Runnable,
TimerTask>();
-    
+
     public Scheduler (String name) {
         this.name = name;
     }
-        
+
     public void executePeriodically(final Runnable task, long period) {
-    	TimerTask timerTask = new SchedulerTimerTask(task);
-        timer.scheduleAtFixedRate(timerTask, period, period);
+        TimerTask timerTask = new SchedulerTimerTask(task);
+        timer.schedule(timerTask, period, period);
         timerTasks.put(task, timerTask);
     }
 
     /*
-     * execute on rough schedual based on termination of last execution. There is no
+     * execute on rough schedule based on termination of last execution. There is no
      * compensation (two runs in quick succession) for delays
      */
     public synchronized void schedualPeriodically(final Runnable task, long period) {
@@ -49,9 +49,9 @@ public final class Scheduler extends Ser
         timer.schedule(timerTask, period, period);
         timerTasks.put(task, timerTask);
     }
-    
+
     public synchronized void cancel(Runnable task) {
-    	TimerTask ticket = timerTasks.remove(task);
+        TimerTask ticket = timerTasks.remove(task);
         if (ticket != null) {
             ticket.cancel();
             timer.purge();//remove cancelled TimerTasks
@@ -59,10 +59,10 @@ public final class Scheduler extends Ser
     }
 
     public synchronized void executeAfterDelay(final Runnable task, long redeliveryDelay)
{
-    	TimerTask timerTask = new SchedulerTimerTask(task);
+        TimerTask timerTask = new SchedulerTimerTask(task);
         timer.schedule(timerTask, redeliveryDelay);
     }
-    
+
     public void shutdown() {
         timer.cancel();
     }
@@ -70,7 +70,7 @@ public final class Scheduler extends Ser
     @Override
     protected synchronized void doStart() throws Exception {
         this.timer = new Timer(name, true);
-        
+
     }
 
     @Override
@@ -78,7 +78,7 @@ public final class Scheduler extends Ser
        if (this.timer != null) {
            this.timer.cancel();
        }
-        
+
     }
 
     public String getName() {



Mime
View raw message