You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by da...@apache.org on 2011/03/05 08:59:00 UTC

svn commit: r1078241 - in /camel/trunk/components/camel-jms/src: main/java/org/apache/camel/component/jms/JmsConfiguration.java test/java/org/apache/camel/component/jms/JmsDurableTopicInvalidConfigurationTest.java

Author: davsclaus
Date: Sat Mar  5 07:58:59 2011
New Revision: 1078241

URL: http://svn.apache.org/viewvc?rev=1078241&view=rev
Log:
CAMEL-3753: Relaxed check for clientId when using durable subscribers.

Removed:
    camel/trunk/components/camel-jms/src/test/java/org/apache/camel/component/jms/JmsDurableTopicInvalidConfigurationTest.java
Modified:
    camel/trunk/components/camel-jms/src/main/java/org/apache/camel/component/jms/JmsConfiguration.java

Modified: camel/trunk/components/camel-jms/src/main/java/org/apache/camel/component/jms/JmsConfiguration.java
URL: http://svn.apache.org/viewvc/camel/trunk/components/camel-jms/src/main/java/org/apache/camel/component/jms/JmsConfiguration.java?rev=1078241&r1=1078240&r2=1078241&view=diff
==============================================================================
--- camel/trunk/components/camel-jms/src/main/java/org/apache/camel/component/jms/JmsConfiguration.java (original)
+++ camel/trunk/components/camel-jms/src/main/java/org/apache/camel/component/jms/JmsConfiguration.java Sat Mar  5 07:58:59 2011
@@ -813,9 +813,6 @@ public class JmsConfiguration implements
             container.setDurableSubscriptionName(durableSubscriptionName);
             container.setSubscriptionDurable(true);
         }
-        if (durableSubscriptionName != null && clientId == null) {
-            throw new IllegalArgumentException("ClientId must be configured when subscription is durable for " + endpoint);
-        }
         if (clientId != null) {
             container.setClientId(clientId);
         }