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

Posted to commits@camel.apache.org by ge...@apache.org on 2008/09/22 11:56:08 UTC

svn commit: r697765 - /activemq/camel/trunk/camel-core/src/main/java/org/apache/camel/processor/StreamResequencer.java

Author: gertv
Date: Mon Sep 22 02:56:08 2008
New Revision: 697765

URL: http://svn.apache.org/viewvc?rev=697765&view=rev
Log:
CAMEL-126: Fixing StreamResequencer.doStop() -- stopping services correctly

Modified:
    activemq/camel/trunk/camel-core/src/main/java/org/apache/camel/processor/StreamResequencer.java

Modified: activemq/camel/trunk/camel-core/src/main/java/org/apache/camel/processor/StreamResequencer.java
URL: http://svn.apache.org/viewvc/activemq/camel/trunk/camel-core/src/main/java/org/apache/camel/processor/StreamResequencer.java?rev=697765&r1=697764&r2=697765&view=diff
==============================================================================
--- activemq/camel/trunk/camel-core/src/main/java/org/apache/camel/processor/StreamResequencer.java (original)
+++ activemq/camel/trunk/camel-core/src/main/java/org/apache/camel/processor/StreamResequencer.java Mon Sep 22 02:56:08 2008
@@ -148,8 +148,10 @@
 
     @Override
     protected void doStop() throws Exception {
-        ServiceHelper.startServices(consumer, processor);
+        // let's stop everything in the reverse order
+        // no need to stop the worker thread -- it will stop automatically when this service is stopped
         engine.stop();
+        ServiceHelper.stopServices(consumer, processor);
     }
 
     /**