Author: aconway
Date: Wed Oct 24 14:33:16 2012
New Revision: 1401710
URL: http://svn.apache.org/viewvc?rev=1401710&view=rev
Log:
Bug-869420 - QPID-4393: BrokerReplicator should call queue->bind, not
exchange->bind
In BrokerReplicator, there are a few places that it calls
exchange->bind(). We're seeing some incorrect behavior during failover testing
(e.g. an exchange shows that it has 5 bindings when there is really only 1) and
we think we've narrowed it down to this. Instead of calling exchange->bind(), it
should call queue->bind(). If you look at qpid::broker::Broker::createObject(),
you'll see that this method calls queue->bind() to bind a queue to an exchange.
Modified:
qpid/trunk/qpid/cpp/src/qpid/ha/BrokerReplicator.cpp
Modified: qpid/trunk/qpid/cpp/src/qpid/ha/BrokerReplicator.cpp
URL:
http://svn.apache.org/viewvc/qpid/trunk/qpid/cpp/src/qpid/ha/BrokerReplicator.cpp?rev=1401710&r1=1401709&r2=1401710&view=diff
==============================================================================
--- qpid/trunk/qpid/cpp/src/qpid/ha/BrokerReplicator.cpp (original)
+++ qpid/trunk/qpid/cpp/src/qpid/ha/BrokerReplicator.cpp Wed Oct 24 14:33:16
2012
@@ -628,7 +628,7 @@ void BrokerReplicator::doResponseBind(Va
<< " key:" << key);
framing::FieldTable args;
qpid::amqp_0_10::translate(asMapVoid(values[ARGUMENTS]), args);
- exchange->bind(queue, key, &args);
+ queue->bind(exchange, key, args);
}
}
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
