Re: Slony Issue Fix in pgAdmin

Поиск
Список
Период
Сортировка
От Neel Patel
Тема Re: Slony Issue Fix in pgAdmin
Дата
Msg-id CAMcbDBE5VEzzLOG3mqNkNgHU3gHUMcEF4RA2sOsSEXPS7=7v=g@mail.gmail.com
обсуждение исходный текст
Ответ на Re: Slony Issue Fix in pgAdmin  (Dave Page <dave.page@enterprisedb.com>)
Ответы Re: Slony Issue Fix in pgAdmin  (Dave Page <dave.page@enterprisedb.com>)
Список pgadmin-hackers
Hi Dave,

Thanks for the comment.

We changed the logic as below.

if ((data->majorVer == 2 && data->minorVer >= 2) || (data->majorVer > 2))
{
      // Execute new query
}
else
{
     // Execute old query
}

Please find the updated patch file in attachment and let me know for comments.

Thanks,
Neel Patel


On Wed, Sep 25, 2013 at 7:50 PM, Dave Page <dave.page@enterprisedb.com> wrote:
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 по дате отправления:

Предыдущее
От: Dave Page
Дата:
Сообщение: Re: Slony Issue Fix in pgAdmin
Следующее
От: Dave Page
Дата:
Сообщение: pgAdmin III commit: Fix ordering of object types in the display options