----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/4846/#review7168 -----------------------------------------------------------
Approach seems reasonable to me. Might be worth checking behaviour for push routes as well, as they are a little ugly/odd. /trunk/qpid/cpp/src/qpid/broker/Link.cpp <https://reviews.apache.org/r/4846/#comment15823> No, that will be done automatically when the session ends. - Gordon On 2012-04-23 20:57:48, Kenneth Giusti wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/4846/ > ----------------------------------------------------------- > > (Updated 2012-04-23 20:57:48) > > > Review request for qpid, Alan Conway and Gordon Sim. > > > Summary > ------- > > Still a WIP, but I wanted early feedback as I'm not too experienced with the > subscription management code involved (completely stolen from Alan). > > This patch allows the Link to subscribe to the remote broker's amq.failover > exchange - if it exists. This allows the Link to be updated dynamically > should the remote broker be part of a cluster, and the cluster membership > changes. > > Light testing against a cluster confirms that this patch fixes qpid-3963. > Testing against a non-cluster remote causes the remote to log the following > error, but otherwise behaves ok: > > 2012-04-23 16:45:27 error Execution exception: not-found: Exchange not found: > amq.failover (../../../qpid/cpp/src/qpid/broker/ExchangeRegistry.cpp:101) > > > This addresses bug qpid-3963. > https://issues.apache.org/jira/browse/qpid-3963 > > > Diffs > ----- > > /trunk/qpid/cpp/src/qpid/broker/Link.h 1329301 > /trunk/qpid/cpp/src/qpid/broker/Link.cpp 1329301 > > Diff: https://reviews.apache.org/r/4846/diff > > > Testing > ------- > > minimal. > > > Thanks, > > Kenneth > >
