diff options
| author | Gordon Sim <gsim@apache.org> | 2012-04-11 12:03:58 +0000 |
|---|---|---|
| committer | Gordon Sim <gsim@apache.org> | 2012-04-11 12:03:58 +0000 |
| commit | eda721c82ea5995b73b65b1bd1540a45c5f61787 (patch) | |
| tree | eb108205e2b365ad56c85ad8cbb8efa8d626d25f | |
| parent | 9ad9acc58355ae7fade15e0518da9239ea3cf9a2 (diff) | |
| download | qpid-python-eda721c82ea5995b73b65b1bd1540a45c5f61787.tar.gz | |
QPID-3943: Make topic subscriptions unreliable by default
git-svn-id: https://svn.apache.org/repos/asf/qpid/trunk/qpid@1324719 13f79535-47bb-0310-9956-ffa450edef68
| -rw-r--r-- | python/qpid/messaging/driver.py | 6 | ||||
| -rw-r--r-- | tests/src/py/qpid_tests/broker_0_10/new_api.py | 2 |
2 files changed, 6 insertions, 2 deletions
diff --git a/python/qpid/messaging/driver.py b/python/qpid/messaging/driver.py index 0358659111..3cb62d75c9 100644 --- a/python/qpid/messaging/driver.py +++ b/python/qpid/messaging/driver.py @@ -226,7 +226,11 @@ class LinkIn: def do_link(self, sst, rcv, _rcv, type, subtype, action): link_opts = _rcv.options.get("link", {}) - reliability = link_opts.get("reliability", "at-least-once") + if type == "topic": + default_reliability = "unreliable" + else: + default_reliability = "at-least-once" + reliability = link_opts.get("reliability", default_reliability) declare = link_opts.get("x-declare", {}) subscribe = link_opts.get("x-subscribe", {}) acq_mode = acquire_mode.pre_acquired diff --git a/tests/src/py/qpid_tests/broker_0_10/new_api.py b/tests/src/py/qpid_tests/broker_0_10/new_api.py index 5e03a7b487..05c4815e57 100644 --- a/tests/src/py/qpid_tests/broker_0_10/new_api.py +++ b/tests/src/py/qpid_tests/broker_0_10/new_api.py @@ -57,7 +57,7 @@ class GeneralTests(Base): sess2 = self.setup_session() tx = sess1.sender("amq.direct/key") - rx_main = sess1.receiver("amq.direct/key;{link:{x-declare:{alternate-exchange:'amq.fanout'}}}") + rx_main = sess1.receiver("amq.direct/key;{link:{reliability:at-least-once,x-declare:{alternate-exchange:'amq.fanout'}}}") rx_alt = sess2.receiver("amq.fanout") rx_alt.capacity = 10 |