> On April 11, 2013, 4:34 p.m., Jason Dillaman wrote:
> > /trunk/qpid/doc/book/src/cpp-broker/Active-Passive-Cluster.xml, line 269
> > <https://reviews.apache.org/r/10376/diff/2/?file=280440#file280440line269>
> >
> >     New behavior supports 'none' keyword and does not default to the broker 
> > url
> 
> Alan Conway wrote:
>     What was the use-case for "none" again?

It's original intention was to match the behavior of the 'known-hosts-url' 
parameter to indicate that the URL should not be defaulted and instead be left 
blank.  However, given that the HA plugin no longer defaults the URL when the 
parameter is not provided, I think it is no longer needed.


- Jason


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


On April 11, 2013, 4:04 p.m., Alan Conway wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/10376/
> -----------------------------------------------------------
> 
> (Updated April 11, 2013, 4:04 p.m.)
> 
> 
> Review request for qpid, Andy Goldstein, Jason Dillaman, mick goulish, Pavel 
> Moravec, and Justin Ross.
> 
> 
> Description
> -------
> 
> NO-JIRA: HA updates to migration guide.
> 
> git-svn-id: https://svn.apache.org/repos/asf/qpid/trunk@1466933 
> 13f79535-47bb-0310-9956-ffa450edef68
> 
> 
> Diffs
> -----
> 
>   /trunk/qpid/cpp/README-HA.txt 1466895 
>   /trunk/qpid/doc/book/src/cpp-broker/Active-Passive-Cluster.xml 1466895 
> 
> Diff: https://reviews.apache.org/r/10376/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Alan Conway
> 
>

Reply via email to