svn commit: r790922 - /camel/trunk/camel-core/src/main/java/org/apache/camel/impl/RouteService.java

View: New views
1 Messages — Rating Filter:   Alert me  

svn commit: r790922 - /camel/trunk/camel-core/src/main/java/org/apache/camel/impl/RouteService.java

by davsclaus-2 :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

Author: davsclaus
Date: Fri Jul  3 14:09:06 2009
New Revision: 790922

URL: http://svn.apache.org/viewvc?rev=790922&view=rev
Log:
Fixed a bug where we potential start some consumers again when stopping.

Modified:
    camel/trunk/camel-core/src/main/java/org/apache/camel/impl/RouteService.java

Modified: camel/trunk/camel-core/src/main/java/org/apache/camel/impl/RouteService.java
URL: http://svn.apache.org/viewvc/camel/trunk/camel-core/src/main/java/org/apache/camel/impl/RouteService.java?rev=790922&r1=790921&r2=790922&view=diff
==============================================================================
--- camel/trunk/camel-core/src/main/java/org/apache/camel/impl/RouteService.java (original)
+++ camel/trunk/camel-core/src/main/java/org/apache/camel/impl/RouteService.java Fri Jul  3 14:09:06 2009
@@ -86,12 +86,10 @@
 
         // there is no lifecycyle for routesRemove
 
-        for (Route route : routes) {
-            List<Service> services = route.getServicesForRoute();
-            for (Service service : services) {
-                stopChildService(service);
-            }
-        }
+        // do not stop child services as in doStart
+        // as route.getServicesForRoute() will restart
+        // already stopped services, so we end up starting
+        // stuff when we stop.
     }
 
     protected LifecycleStrategy getLifecycleStrategy() {