[GitHub] activemq-artemis pull request #1596: ARTEMIS-450 Deadlocked queues over AMQP...

[clebertsuconic]

Github user clebertsuconic commented on a diff in the pull request:

    https://github.com/apache/activemq-artemis/pull/1596#discussion_r145416018
  
    --- Diff: 
artemis-commons/src/main/java/org/apache/activemq/artemis/core/server/ActiveMQScheduledComponent.java
 ---
    @@ -251,6 +251,9 @@ public synchronized boolean isStarted() {
        // this will restart the scheduled component upon changes
        private void restartIfNeeded() {
           if (isStarted()) {
    +         // it has already been through an initial delay,
    +         // now we just use the next interval
    +         this.initialDelay = period;
    --- End diff --
    
    Nope. Initial delay should be at the beginnnjng only.  Why would you want a 
further initial delay when it’s not initial any longer.  Just to keep it 
simple.  


---