-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/4846/
-----------------------------------------------------------

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

Reply via email to