You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by ak...@apache.org on 2010/04/02 21:40:50 UTC
svn commit: r930358 -
/camel/trunk/camel-core/src/main/java/org/apache/camel/component/seda/SedaComponent.java
Author: akarpe
Date: Fri Apr 2 19:40:50 2010
New Revision: 930358
URL: http://svn.apache.org/viewvc?rev=930358&view=rev
Log:
CAMEL-2526 - Changed the behavior of the fix to thrown an exception instead of informing/warning the user
Modified:
camel/trunk/camel-core/src/main/java/org/apache/camel/component/seda/SedaComponent.java
Modified: camel/trunk/camel-core/src/main/java/org/apache/camel/component/seda/SedaComponent.java
URL: http://svn.apache.org/viewvc/camel/trunk/camel-core/src/main/java/org/apache/camel/component/seda/SedaComponent.java?rev=930358&r1=930357&r2=930358&view=diff
==============================================================================
--- camel/trunk/camel-core/src/main/java/org/apache/camel/component/seda/SedaComponent.java (original)
+++ camel/trunk/camel-core/src/main/java/org/apache/camel/component/seda/SedaComponent.java Fri Apr 2 19:40:50 2010
@@ -63,12 +63,8 @@ public class SedaComponent extends Defau
protected Endpoint createEndpoint(String uri, String remaining, Map<String, Object> parameters) throws Exception {
int consumers = getAndRemoveParameter(parameters, "concurrentConsumers", Integer.class, 1);
boolean limitConcurrentConsumers = getAndRemoveParameter(parameters, "limitConcurrentConsumers", Boolean.class, true);
- if (limitConcurrentConsumers) {
- consumers = (consumers < maxConcurrentConsumers) ? consumers : maxConcurrentConsumers;
- if (consumers == maxConcurrentConsumers) {
- LOG.info("The limitConcurrentConsumers flag in set to true. Concurrent Consumers cannot be set at a value greater than " + maxConcurrentConsumers);
- LOG.info("Concurrent Consumers set to " + maxConcurrentConsumers);
- }
+ if ((limitConcurrentConsumers) && (consumers > maxConcurrentConsumers)) {
+ throw new IllegalArgumentException("The limitConcurrentConsumers flag in set to true. Concurrent Consumers cannot be set at a value greater than " + maxConcurrentConsumers);
}
SedaEndpoint answer = new SedaEndpoint(uri, this, createQueue(uri, parameters), consumers);
answer.configureProperties(parameters);