----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10376/#review18853 -----------------------------------------------------------
/branches/0.22/qpid/cpp/README-HA.txt <https://reviews.apache.org/r/10376/#comment39304> a client will only fail over to other URLs if the connection options include multiple URLs - if the client only specifies 1 URL, and it happens to point to a broker that is currently the backup, it will never connect... right? /branches/0.22/qpid/cpp/README-HA.txt <https://reviews.apache.org/r/10376/#comment39305> typo: exaple -> example /branches/0.22/qpid/cpp/README-HA.txt <https://reviews.apache.org/r/10376/#comment39306> or other unknown behaviors /branches/0.22/qpid/doc/book/src/cpp-broker/Active-Passive-Cluster.xml <https://reviews.apache.org/r/10376/#comment39308> assuming the client is configured with a list of urls /branches/0.22/qpid/doc/book/src/cpp-broker/Active-Passive-Cluster.xml <https://reviews.apache.org/r/10376/#comment39309> extra space in between a and cluster - Andy Goldstein On April 9, 2013, 3:57 p.m., Alan Conway wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/10376/ > ----------------------------------------------------------- > > (Updated April 9, 2013, 3:57 p.m.) > > > Review request for qpid, Andy Goldstein, Jason Dillaman, Pavel Moravec, and > Justin Ross. > > > Description > ------- > > This is the first draft of HA migration guide, I'm hoping to get some review > feedback. > > The patch also has some minor updates to the HA chapter of the Qpid broker > book, they're less important for review. > > > Diffs > ----- > > /branches/0.22/qpid/cpp/README-HA.txt PRE-CREATION > /branches/0.22/qpid/doc/book/src/cpp-broker/Active-Passive-Cluster.xml > 1464646 > > Diff: https://reviews.apache.org/r/10376/diff/ > > > Testing > ------- > > > Thanks, > > Alan Conway > >
