----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/5895/#review9054 -----------------------------------------------------------
Ship it! Ship It! - Gordon Sim On July 10, 2012, 7:53 p.m., Alan Conway wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/5895/ > ----------------------------------------------------------- > > (Updated July 10, 2012, 7:53 p.m.) > > > Review request for qpid. > > > Description > ------- > > HA replication links should not use the amq.failover exchange to get updates > reconnect targets. amq.failover provides the client failover list, HA manages > a separate failover list for brokers. Replication links should be using the > broker list, and not allow it to be overwritten by amq.failover updates. > > Review requested for 0.18, 2 line fix. > > > Diffs > ----- > > /trunk/qpid/cpp/src/qpid/ha/Backup.cpp 1359232 > /trunk/qpid/cpp/src/qpid/ha/HaBroker.cpp 1359232 > > Diff: https://reviews.apache.org/r/5895/diff/ > > > Testing > ------- > > > Thanks, > > Alan Conway > >
