Ashesh - did you get a chance to test this?
On Thu, Sep 3, 2009 at 2:37 PM, Ashesh Vashi
<ashesh.vashi@enterprisedb.com> wrote:
> Looks good to me..
> Though not tested..
> I will be able to test it tomorrow.
>
> Dave Page wrote:
>
> On Thu, Sep 3, 2009 at 2:08 PM, Sachin
> Srivastava<sachin.srivastava@enterprisedb.com> wrote:
>
> Dave,
>
> - bool dlgRepCluster::SlonyMaximumVersion(wxString series, long minor)
>
> should be:
>
> bool dlgRepCluster::SlonyMaximumVersion(const wxString&series, long minor)
>
> Done.
>
> - Just reading the patch, it looks like we never try to load xxid*.sql
> any more. That's surely wrong?
>
> Yes. Attached is an updated patch.
>
> Thanks. I've had a quick look, and it looks reasonable to me. Ashesh;
> can you review it as well please?
>
>
--
Dave Page
EnterpriseDB UK: http://www.enterprisedb.com