camel-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From ningji...@apache.org
Subject [6/8] camel git commit: CAMEL-9196: Timeout must be a positive value when calling stop route with a timeout value.
Date Thu, 08 Oct 2015 03:30:27 GMT
CAMEL-9196: Timeout must be a positive value when calling stop route with a timeout value.


Project: http://git-wip-us.apache.org/repos/asf/camel/repo
Commit: http://git-wip-us.apache.org/repos/asf/camel/commit/9e7516e7
Tree: http://git-wip-us.apache.org/repos/asf/camel/tree/9e7516e7
Diff: http://git-wip-us.apache.org/repos/asf/camel/diff/9e7516e7

Branch: refs/heads/camel-2.16.x
Commit: 9e7516e7a4b34803ba517f8435f0b6208c0589a1
Parents: 63437c6
Author: Claus Ibsen <davsclaus@apache.org>
Authored: Wed Oct 7 07:15:36 2015 +0200
Committer: Willem Jiang <willem.jiang@gmail.com>
Committed: Thu Oct 8 11:26:28 2015 +0800

----------------------------------------------------------------------
 .../java/org/apache/camel/impl/DefaultShutdownStrategy.java     | 5 +++++
 1 file changed, 5 insertions(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/camel/blob/9e7516e7/camel-core/src/main/java/org/apache/camel/impl/DefaultShutdownStrategy.java
----------------------------------------------------------------------
diff --git a/camel-core/src/main/java/org/apache/camel/impl/DefaultShutdownStrategy.java b/camel-core/src/main/java/org/apache/camel/impl/DefaultShutdownStrategy.java
index 0657f15..2eac21e 100644
--- a/camel-core/src/main/java/org/apache/camel/impl/DefaultShutdownStrategy.java
+++ b/camel-core/src/main/java/org/apache/camel/impl/DefaultShutdownStrategy.java
@@ -166,6 +166,11 @@ public class DefaultShutdownStrategy extends ServiceSupport implements
ShutdownS
     protected boolean doShutdown(CamelContext context, List<RouteStartupOrder> routes,
long timeout, TimeUnit timeUnit,
                                  boolean suspendOnly, boolean abortAfterTimeout, boolean
forceShutdown) throws Exception {
 
+        // timeout must be a positive value
+        if (timeout <= 0) {
+            throw new IllegalArgumentException("Timeout must be a positive value");
+        }
+
         // just return if no routes to shutdown
         if (routes.isEmpty()) {
             return true;


Mime
View raw message