Re: Slony Issue Fix in pgAdmin - Mailing list pgadmin-hackers

From Dave Page
Subject Re: Slony Issue Fix in pgAdmin
Date
Msg-id CA+OCxozMi-KVZ4uw-bnNjHLKV8_tF+0aKFC+sU=34C0sF0LteQ@mail.gmail.com
Whole thread Raw
In response to Slony Issue Fix in pgAdmin  (Neel Patel <neel.patel@enterprisedb.com>)
Responses Re: Slony Issue Fix in pgAdmin
List pgadmin-hackers
Hi

On Tue, Sep 24, 2013 at 12:24 PM, Neel Patel
<neel.patel@enterprisedb.com> wrote:
> Hi Dave,
>
> Please find the attached patch file for fix of below slony issue in pgAdmin.
>
> Steps to reproduce:-
>
> - Install PostgreSQL 9.3 and Slony version 2.2.0.
> - Open pgAdmin
> - Create new Schema and give cluster name in "Use Slony" option.
> - Click OK and below error will come.
>
>
> ##################################################################
>
> ERROR: function _edb_cluster.ddlscript_prepare(integer, integer) does not
> exist
> LINE 1: SELECT _edb_cluster.ddlscript_prepare(1, -1);
>
> ##################################################################
>
>
> Solution:-
>
> As we are using ddlscript_prepare() statement to replicate the new schema to
> slave and from new Slony version 2.2.0 this function no more supported and
> removed so as per the document we have to use ddlcapture() method instead of
> ddlscript_prepare(). Also ddlscript_complete() method arguments got changed
> so that also fixed.
>
> Please review it and let me know for any modifications.

This seems wrong to me:

if (data->majorVer >= 2 && data->minorVer >= 2)

We'll get the old query with Slony 3.0 and 3.1 I think.

--
Dave Page
Chief Architect, Tools & Installers
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake


pgadmin-hackers by date:

Previous
From: Dave Page
Date:
Subject: Re: compiling pgadmin on OS X
Next
From: Neel Patel
Date:
Subject: Re: Slony Issue Fix in pgAdmin