svn commit: r1675286 - /sling/trunk/bundles/extensions/event/src/main/java/org/apache/sling/event/impl/jobs/timed/TimedEventSender.java

[cz...@apache.org]

Author: cziegeler
Date: Wed Apr 22 09:13:20 2015
New Revision: 1675286

URL: http://svn.apache.org/r1675286
Log:
Fix comment

Modified:
    sling/trunk/bundles/extensions/event/src/main/java/org/apache/sling/event/impl/jobs/timed/TimedEventSender.java

Modified: sling/trunk/bundles/extensions/event/src/main/java/org/apache/sling/event/impl/jobs/timed/TimedEventSender.java
URL: http://svn.apache.org/viewvc/sling/trunk/bundles/extensions/event/src/main/java/org/apache/sling/event/impl/jobs/timed/TimedEventSender.java?rev=1675286&r1=1675285&r2=1675286&view=diff
==============================================================================
--- sling/trunk/bundles/extensions/event/src/main/java/org/apache/sling/event/impl/jobs/timed/TimedEventSender.java (original)
+++ sling/trunk/bundles/extensions/event/src/main/java/org/apache/sling/event/impl/jobs/timed/TimedEventSender.java Wed Apr 22 09:13:20 2015
@@ -256,9 +256,7 @@ public class TimedEventSender
 
                     event = null;
                 } else if (NotificationConstants.TOPIC_JOB_FINISHED.equals(event.getTopic())){
-                    // stopScheduling() puts this event on the queue, but the intention is unclear to me.
-                    // as the threadStarted flag ensures the background thread is only started once, we must not stop
-                    // the thread, otherwise its never started again upon topology changes.
+                    // stopScheduling() puts this event on the queue, to wake up this thread
                     event = null;
                 } else {
                     // to ensure the event is reset to null in any case, in order to take from the queue again