You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by ni...@apache.org on 2015/10/08 05:30:27 UTC
[6/8] camel git commit: CAMEL-9196: Timeout must be a positive value
when calling stop route with a timeout value.
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 <da...@apache.org>
Authored: Wed Oct 7 07:15:36 2015 +0200
Committer: Willem Jiang <wi...@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;