[QPID-4393] HA BrokerReplicator should call queue->bind, not exchange->bind

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 0.19
• Component/s: C++ Clustering
• Assignee: Alan Conway
• Reporter: Alan Conway
• Created: 23/Oct/12 21:04
• Updated: 29/Jul/13 18:53
• Resolved: 24/Oct/12 14:35

Description

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.

(Note that exchange->unbind() appears to be the correct operation for removing a binding, so this shouldn't need to be changed.)