Author: aconway
Date: Tue May 29 13:15:04 2012
New Revision: 1343710
URL: http://svn.apache.org/viewvc?rev=1343710&view=rev
Log:
QPID-3603: Removed isSelf check in Backup, was returning false positives.
To be restored once the problem is fixed.
Modified:
qpid/trunk/qpid/cpp/src/qpid/ha/Backup.cpp
Modified: qpid/trunk/qpid/cpp/src/qpid/ha/Backup.cpp
URL:
http://svn.apache.org/viewvc/qpid/trunk/qpid/cpp/src/qpid/ha/Backup.cpp?rev=1343710&r1=1343709&r2=1343710&view=diff
==============================================================================
--- qpid/trunk/qpid/cpp/src/qpid/ha/Backup.cpp (original)
+++ qpid/trunk/qpid/cpp/src/qpid/ha/Backup.cpp Tue May 29 13:15:04 2012
@@ -57,6 +57,8 @@ bool Backup::isSelf(const Address& a) co
}
Url Backup::linkUrl(const Url& brokers) const {
+ return brokers;
+ /** FIXME aconway 2012-05-29: Problems with self-test, false positives.
// linkUrl contains only the addresses of *other* brokers, not this one.
Url url;
for (Url::const_iterator i = brokers.begin(); i != brokers.end(); ++i)
@@ -64,6 +66,7 @@ Url Backup::linkUrl(const Url& brokers)
if (url.empty()) throw Url::Invalid("HA Backup failover URL is empty");
QPID_LOG(debug, logPrefix << "Backup failover URL (excluding self): " <<
url);
return url;
+ */
}
void Backup::initialize(const Url& brokers) {
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@qpid.apache.org
For additional commands, e-mail: commits-h...@qpid.apache.org
