diff options
| author | Rajith Muditha Attapattu <rajith@apache.org> | 2011-03-07 22:02:24 +0000 |
|---|---|---|
| committer | Rajith Muditha Attapattu <rajith@apache.org> | 2011-03-07 22:02:24 +0000 |
| commit | 7a15f50220055075fb4935a601909d4402667462 (patch) | |
| tree | e09d4b42b18eae233d54ad09b68605b41c01c503 | |
| parent | ab3a8d6bf4f22b7271047bbf4377fdbb9135bbeb (diff) | |
| download | qpid-python-7a15f50220055075fb4935a601909d4402667462.tar.gz | |
QPID-2732
Set the default reliability modes for Topics and Queues.
The code now throws an exception if at-least-once is specified for Topics as we don't support it properly atm.
git-svn-id: https://svn.apache.org/repos/asf/qpid/trunk@1078971 13f79535-47bb-0310-9956-ffa450edef68
| -rw-r--r-- | qpid/java/client/src/main/java/org/apache/qpid/client/AMQSession_0_10.java | 16 | ||||
| -rw-r--r-- | qpid/java/client/src/main/java/org/apache/qpid/client/messaging/address/Link.java | 7 |
2 files changed, 19 insertions, 4 deletions
diff --git a/qpid/java/client/src/main/java/org/apache/qpid/client/AMQSession_0_10.java b/qpid/java/client/src/main/java/org/apache/qpid/client/AMQSession_0_10.java index 6b4f6c1640..7398a89e9a 100644 --- a/qpid/java/client/src/main/java/org/apache/qpid/client/AMQSession_0_10.java +++ b/qpid/java/client/src/main/java/org/apache/qpid/client/AMQSession_0_10.java @@ -1173,6 +1173,22 @@ public class AMQSession_0_10 extends AMQSession<BasicMessageConsumer_0_10, Basic int type = resolveAddressType(dest); + if (type == AMQDestination.QUEUE_TYPE && + dest.getLink().getReliability() == Reliability.UNSPECIFIED) + { + dest.getLink().setReliability(Reliability.AT_LEAST_ONCE); + } + else if (type == AMQDestination.TOPIC_TYPE && + dest.getLink().getReliability() == Reliability.UNSPECIFIED) + { + dest.getLink().setReliability(Reliability.UNRELIABLE); + } + else if (type == AMQDestination.TOPIC_TYPE && + dest.getLink().getReliability() == Reliability.AT_LEAST_ONCE) + { + throw new AMQException("AT-LEAST-ONCE is not yet supported for Topics"); + } + switch (type) { case AMQDestination.QUEUE_TYPE: diff --git a/qpid/java/client/src/main/java/org/apache/qpid/client/messaging/address/Link.java b/qpid/java/client/src/main/java/org/apache/qpid/client/messaging/address/Link.java index f10c6f7fca..5f97d625b4 100644 --- a/qpid/java/client/src/main/java/org/apache/qpid/client/messaging/address/Link.java +++ b/qpid/java/client/src/main/java/org/apache/qpid/client/messaging/address/Link.java @@ -20,8 +20,7 @@ */ package org.apache.qpid.client.messaging.address; -import static org.apache.qpid.client.messaging.address.Link.Reliability.UNRELIABLE; -import static org.apache.qpid.client.messaging.address.Link.Reliability.AT_LEAST_ONCE; +import static org.apache.qpid.client.messaging.address.Link.Reliability.UNSPECIFIED; import java.util.HashMap; import java.util.Map; @@ -32,7 +31,7 @@ public class Link { public enum FilterType { SQL92, XQUERY, SUBJECT } - public enum Reliability { UNRELIABLE, AT_MOST_ONCE, AT_LEAST_ONCE, EXACTLY_ONCE } + public enum Reliability { UNRELIABLE, AT_MOST_ONCE, AT_LEAST_ONCE, EXACTLY_ONCE, UNSPECIFIED } protected String name; protected String _filter; @@ -43,7 +42,7 @@ public class Link protected int _producerCapacity = 0; protected Node node; protected Subscription subscription; - protected Reliability reliability = AT_LEAST_ONCE; + protected Reliability reliability = UNSPECIFIED; public Reliability getReliability() { |