diff options
| author | Alan Conway <aconway@apache.org> | 2012-07-10 20:07:16 +0000 |
|---|---|---|
| committer | Alan Conway <aconway@apache.org> | 2012-07-10 20:07:16 +0000 |
| commit | bdb3e4e56a1c3074193217499adb52ee10170989 (patch) | |
| tree | fc937f90b081e773ea9ff7a5a6a89ca289e98f3a /qpid/cpp | |
| parent | 52d262d135090ccd1d9c8a35db659f88b7a3e748 (diff) | |
| download | qpid-python-bdb3e4e56a1c3074193217499adb52ee10170989.tar.gz | |
QPID-4128: HA should not use amq.failover for replication links
HA replication links should not use the amq.failover exchange to get updates
reconnect targets. amq.failover provides the client failover list, HA manages a
separate failover list for brokers. Replication links should be using the broker
list, and not allow it to be overwritten by amq.failover updates.
Review requested for 0.18, 2 line fix.
git-svn-id: https://svn.apache.org/repos/asf/qpid/trunk@1359879 13f79535-47bb-0310-9956-ffa450edef68
Diffstat (limited to 'qpid/cpp')
| -rw-r--r-- | qpid/cpp/src/qpid/ha/Backup.cpp | 2 | ||||
| -rw-r--r-- | qpid/cpp/src/qpid/ha/HaBroker.cpp | 3 |
2 files changed, 3 insertions, 2 deletions
diff --git a/qpid/cpp/src/qpid/ha/Backup.cpp b/qpid/cpp/src/qpid/ha/Backup.cpp index 8ffe411c91..4ec8af588c 100644 --- a/qpid/cpp/src/qpid/ha/Backup.cpp +++ b/qpid/cpp/src/qpid/ha/Backup.cpp @@ -82,7 +82,7 @@ void Backup::initialize(const Url& brokers) { url[0].host, url[0].port, protocol, false, // durable settings.mechanism, settings.username, settings.password, - true); // amq.failover + false); // no amq.failover - don't want to use client URL. { sys::Mutex::ScopedLock l(lock); link = result.first; diff --git a/qpid/cpp/src/qpid/ha/HaBroker.cpp b/qpid/cpp/src/qpid/ha/HaBroker.cpp index 1728ed6858..858f04e198 100644 --- a/qpid/cpp/src/qpid/ha/HaBroker.cpp +++ b/qpid/cpp/src/qpid/ha/HaBroker.cpp @@ -175,7 +175,8 @@ Manageable::status_t HaBroker::ManagementMethod (uint32_t methodId, Args& args, broker::QPID_NAME_PREFIX + string("ha.link.") + uuid.str(), url[0].host, url[0].port, protocol, false, // durable - settings.mechanism, settings.username, settings.password); + settings.mechanism, settings.username, settings.password, + false); // no amq.failover - don't want to use client URL. boost::shared_ptr<broker::Link> link = result.first; link->setUrl(url); // Create a queue replicator |