Re: pgAdmin 1.18.0 + slony-I 2.2.0 + PG 9.3

From: Neel Patel <neel(dot)patel(at)enterprisedb(dot)com>
To: Ashesh Vashi <ashesh(dot)vashi(at)enterprisedb(dot)com>
Cc: Dave Page <dave(dot)page(at)enterprisedb(dot)com>, Sven Meirsman <sven(dot)meirsman(at)gmail(dot)com>, pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>, Guillaume Lelarge <guillaume(at)lelarge(dot)info>, pgAdmin Support <pgadmin-support(at)postgresql(dot)org>
Subject: Re: pgAdmin 1.18.0 + slony-I 2.2.0 + PG 9.3
Date: 2013-11-11 13:16:18
Message-ID: CAMcbDBG2KnMsdqu9cCo7Ab2Frd-q-w3ybkKuMuY5XE4xNSwCxw@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers pgadmin-support

Hi Ashesh,

Please find the updated patch with below fix.

-- Fix for slony version 1.2.23 with PG 9.0
-- Picking the correct version of slony1_base.xxx.sql
-- Fix the error "no schema has been selected to create in". It gives error
after deleting the slony cluster and again create the new slony cluster.
-- Fix for function storeNode() API which is accepting three arguments in
slony version 1.1 and 1.2 and two arguments in
slony version 2.0 and above.

Please let me know for any comments.

Thanks,
Neel Patel

On Thu, Oct 31, 2013 at 9:41 AM, Neel Patel <neel(dot)patel(at)enterprisedb(dot)com>wrote:

> Hi,
>
> Thanks Ashesh.
>
> I will check and let you know.
>
> Thanks,
> Neel Patel
>
>
> On Wed, Oct 30, 2013 at 5:19 PM, Ashesh Vashi <
> ashesh(dot)vashi(at)enterprisedb(dot)com> wrote:
>
>> Hi Neel,
>>
>> I test the patch with following slony-I versions:
>> 1.2.27 (not working with PG 9.0)
>> 2.0.x (not picking correct slony1_base.xxx.sql - as per comment, it
>> should pick slony1_base.v84.sql with PG 8.4 and above, but picking
>> slony1_base.v83.sql)
>>
>> I have not tested further.
>> Can you please take a look at it?
>>
>>
>>
>> On Tue, Oct 29, 2013 at 3:38 PM, Dave Page <dave(dot)page(at)enterprisedb(dot)com>wrote:
>>
>>> Ashesh, can you look at this one as well please? Thanks.
>>>
>>> On Tue, Oct 22, 2013 at 8:34 AM, Neel Patel <neel(dot)patel(at)enterprisedb(dot)com>
>>> wrote:
>>> > Hi Dave,
>>> >
>>> > We have fixed the below two issues to support in Slony version 2.2.0
>>> with PG
>>> > 9.3 in pgAdmin.
>>> >
>>> > Issue 1:-
>>> >
>>> > Bug to reproduce:-
>>> >
>>> > - Install PG 9.3 and slony 2.2.0
>>> > - Right click on database and click on option "New slony-I cluster..."
>>> > - In the dialog we are able to see the error in status bar saying
>>> "Slony-I
>>> > creation scripts not available; only joining possible"
>>> >
>>> > Solution :-
>>> >
>>> > As per the above conversation in new version of slony 2.2.0
>>> installation
>>> > name of the sql files got changed so it is not able to find the current
>>> > slony version.
>>> >
>>> >
>>> > Issue 2:-
>>> >
>>> > "storenode" API contains three argument in slony version 1.1 and 1.2
>>> but in
>>> > slony version 2.0 onwards it contains only two arguments so that issue
>>> also
>>> > fixed.
>>> >
>>> >
>>> > As per our knowledge we have tested this patch in slony 2.2.0 with PG
>>> 9.3.
>>> >
>>> > Can anyone please test this patch and let us know if any bugs.
>>> >
>>> >
>>> > Thanks,
>>> > Neel Patel
>>> >
>>> >
>>> >
>>> >
>>> >
>>> > On Fri, Oct 11, 2013 at 5:17 PM, Neel Patel <
>>> neel(dot)patel(at)enterprisedb(dot)com>
>>> > wrote:
>>> >>
>>> >> Hi Sven,
>>> >>
>>> >> No idea regarding the next release.
>>> >>
>>> >> Thanks,
>>> >> Neel Patel
>>> >>
>>> >>
>>> >> On Fri, Oct 11, 2013 at 11:46 AM, Sven Meirsman <
>>> sven(dot)meirsman(at)gmail(dot)com>
>>> >> wrote:
>>> >>>
>>> >>> Hi,
>>> >>>
>>> >>> Thanks for the feedback. I'm looking forward to this update. Any
>>> idea on
>>> >>> when it will be released?
>>> >>>
>>> >>> regards,
>>> >>> Sven
>>> >>>
>>> >>>
>>> >>> 2013/10/11 Neel Patel <neel(dot)patel(at)enterprisedb(dot)com>
>>> >>>>
>>> >>>> Hi Sven,
>>> >>>>
>>> >>>> Sorry for the late replay, we have checked and it is a bug in
>>> pgAdmin
>>> >>>> because the name of the sql files got changed in new slony version
>>> ( v2.2.0
>>> >>>> ) which is used by the pgAdmin.
>>> >>>>
>>> >>>> Issue :-
>>> >>>>
>>> >>>> When we install the slony ( version less than 2.2.0 ) then it will
>>> >>>> install below sql files in C:\Program Files\PostgreSQL\9.3\share
>>> folder.
>>> >>>>
>>> >>>> slony1_base.sql
>>> >>>> slony1_base.v83.sql
>>> >>>> slony1_base.v84.sql
>>> >>>> slony1_funcs.sql
>>> >>>> slony1_funcs.v83.sql
>>> >>>> slony1_funcs.v84.sql
>>> >>>>
>>> >>>> Now in New version of slony 2.2.0 onwards the name of the above sql
>>> >>>> files got changed as below.
>>> >>>>
>>> >>>> slony1_base.2.2.0.sql
>>> >>>> slony1_base.v83.2.2.0.sql
>>> >>>> slony1_base.v84.2.2.0.sql
>>> >>>> slony1_funcs.2.2.0.sql
>>> >>>> slony1_funcs.v83.2.2.0.sql
>>> >>>> slony1_funcs.v84.2.2.0.sql
>>> >>>>
>>> >>>> As we are using above sql files names in pgAdmin to create the new
>>> slony
>>> >>>> clusters so we need to change in pgAdmin also.
>>> >>>>
>>> >>>> We are working on this issue and submit the patch once its done.
>>> >>>>
>>> >>>> Thanks,
>>> >>>> Neel Patel
>>> >>>>
>>> >>>>
>>> >>>>
>>> >>>> On Mon, Sep 30, 2013 at 2:05 AM, Guillaume Lelarge
>>> >>>> <guillaume(at)lelarge(dot)info> wrote:
>>> >>>>>
>>> >>>>> On Sun, 2013-09-29 at 22:29 +0200, Sven Meirsman wrote:
>>> >>>>> > Sorry, My mistake.
>>> >>>>> >
>>> >>>>> > 'Binary Paths' is the section you choose at the left of the
>>> screen.
>>> >>>>> > It
>>> >>>>> > contains several paths, one 'PG Bin Path' is directing to my
>>> postgres
>>> >>>>> > BIN
>>> >>>>> > folder like you suggest. It is the Slony-I path that contains the
>>> >>>>> > path to
>>> >>>>> > the postgres SHARE folder where the slony scripts reside.
>>> >>>>> >
>>> >>>>>
>>> >>>>> Oh OK, you're right.
>>> >>>>>
>>> >>>>> On the Slony issue, I see that Dave commited a patch from Neel
>>> Patel
>>> >>>>> last thursday. This patch commit log is: "Ensure DDLScript will
>>> work
>>> >>>>> with Slony 2.2+ which has a new SQL level API.". So I guess this is
>>> >>>>> what
>>> >>>>> you're looking for. This will be in the next minor release
>>> (1.18.1),
>>> >>>>> which should be available on thursday 10th october.
>>> >>>>>
>>> >>>>>
>>> >>>>> --
>>> >>>>> Guillaume
>>> >>>>> http://blog.guillaume.lelarge.info
>>> >>>>> http://www.dalibo.com
>>> >>>>>
>>> >>>>>
>>> >>>>>
>>> >>>>> --
>>> >>>>> Sent via pgadmin-support mailing list (
>>> pgadmin-support(at)postgresql(dot)org)
>>> >>>>> To make changes to your subscription:
>>> >>>>> http://www.postgresql.org/mailpref/pgadmin-support
>>> >>>>
>>> >>>>
>>> >>>
>>> >>
>>> >
>>>
>>>
>>>
>>> --
>>> Dave Page
>>> Chief Architect, Tools & Installers
>>> EnterpriseDB: http://www.enterprisedb.com
>>> The Enterprise PostgreSQL Company
>>>
>>> Blog: http://pgsnake.blogspot.com
>>> Twitter: @pgsnake
>>>
>>
>>
>>
>> --
>> --
>>
>> Thanks & Regards,
>>
>> Ashesh Vashi
>> EnterpriseDB INDIA: Enterprise PostgreSQL Company<http://www.enterprisedb.com>
>>
>>
>>
>> *http://www.linkedin.com/in/asheshvashi*<http://www.linkedin.com/in/asheshvashi>
>>
>
>

Attachment Content-Type Size
Slony_Fix.patch application/octet-stream 11.2 KB

In response to

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message Christoph Zwerschke 2013-11-14 19:04:57 Fix for a problem with auto reconnection
Previous Message Dinesh Kumar 2013-11-11 06:57:29 Re: Patch for EDB binary path not set by default.

Browse pgadmin-support by date

  From Date Subject
Next Message Anton Bolotinsky 2013-11-11 18:20:28 Urgent help!
Previous Message Doug Easterbrook 2013-11-09 18:25:11 Mavericks: error in console log with pgadmin 1.18.1