On Wed, 2006-11-08 at 07:00 -0800, Darcy Buskermolen wrote:
> On Tuesday 07 November 2006 15:03, Jeff Davis wrote:
> > I run slony and have escape_string_warning = on. Is there any interest
> > in preventing slony from using queries that cause that warning?
> >
> > We could either detect backends >= 8.1 and use E'' strings, or use
> > PQexecParams instead.
> >
> > Is there a reason that's not done? If I wrote such a patch, would it be
> > accepted, or would it be needless code complexity?
> 
> We could take the same approach with this as we did with standards conformant 
> strings, which is we set the guc's on the sesssions between the nodes.  Would 
> that fix your issue?
> 
> Introduced in  rev 1.2.3 of  slony1-engine/src/slon/dbutils.c

Sounds good to me.

However, I could see how someone would be confused if they explicitly
set the warnings on and then they don't get the warnings. I can't think
of a particular problem that would cause, but I thought I'd bring it up.

Thanks,
	Jeff Davis