[prev in list] [next in list] [prev in thread] [next in thread] 

List:       slony1-general
Subject:    Re: [Slony1-general] Problem with 1.2.8 and EXECUTE SCRIPT with
From:       Alan Hodgson <ahodgson () simkin ! ca>
Date:       2007-03-29 21:46:39
Message-ID: 200703291446.39520 () hal ! medialogik ! com
[Download RAW message or body]

On Tuesday 27 March 2007 17:45, Steve Singer <ssinger_pg@sympatico.ca> 
wrote:
> I don't know what workarounds are going to be options for you compared
> with running a patched version of slony but if you
> need a fix right away you could consider patching your version of slony.
>
> The patched attached to
> http://lists.slony.info/pipermail/slony1-patches/2007-March/000005.html
>
> should address the problem your describing.  I assume it will be
> included with the next 1.2.x release.

The patch referenced above, with the very small fix I posted earlier 
to -patches, completely fixes this problem.  Hopefully it will get reviewed 
and make its way into 1.2.10.

As I suspected, my problem was due to ddlScript_prepare_int not running 
alterTableRestore against tables (because it checked for set membership), 
but ddlScript_complete_int trying to run alterTableForReplication (because 
it doesn't). However, I misunderstood what should be the normal contents of 
sl_table. As it turns out, there is no need for changes to the SQL 
functions if used with Steve's patch. There are some checks in 
ddlScript_prepare_int that are made redundant and could be removed, but 
they no longer hurt anything.

Steve - thank you very much for your help.

-- 
Opportunity is missed by most people because it is dressed in overalls and
looks like work. - Thomas Edison
_______________________________________________
Slony1-general mailing list
Slony1-general@lists.slony.info
http://lists.slony.info/mailman/listinfo/slony1-general
[prev in list] [next in list] [prev in thread] [next in thread]