Re: Re: [BUGS] BUG #4027: backslash escapingnotdisabledinplpgsql

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: "Kevin Grittner" <Kevin(dot)Grittner(at)wicourts(dot)gov>
Cc: pgsql-hackers(at)postgresql(dot)org, "Andrew Gierth" <andrew(at)tao11(dot)riddles(dot)org(dot)uk>
Subject: Re: Re: [BUGS] BUG #4027: backslash escapingnotdisabledinplpgsql
Date: 2009-04-10 21:24:17
Message-ID: 1432.1239398657@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-bugs pgsql-hackers

"Kevin Grittner" <Kevin(dot)Grittner(at)wicourts(dot)gov> writes:
> Well, that's a change I'm arguing for. That would require both the
> plpgsql parser change Tom is talking about, and a change to CREATE
> FUNCTION such that there is an implied SET standard_compliant_strings
> FROM CURRENT -- which is something I've suggested a couple times;
> there's been no explicit response to that.

If you want one: it seems like a really bad idea. Aside from the sheer
ugliness of special-casing one particular GUC, it would break existing
pg_dump files, since pg_dump has no idea that its setting of
standard_conforming_strings might influence the behavior of functions
it defines.

I don't actually see that standard_conforming_strings is worse than
search_path or half a dozen other settings that will influence the
semantics of SQL queries. If anything it's less bad than those since
it's less likely to break things silently. The whole topic just
illustrates that "invent a GUC" is not a pain-free solution to handling
definitional conflicts.

regards, tom lane

In response to

Responses

Browse pgsql-bugs by date

  From Date Subject
Next Message Jeff Davis 2009-04-10 21:29:01 Re: possible bug not in open items
Previous Message Tom Lane 2009-04-10 21:16:14 Re: Re: [BUGS] BUG #4027: backslash escapingnotdisabled inplpgsql

Browse pgsql-hackers by date

  From Date Subject
Next Message Tom Lane 2009-04-10 21:25:15 Re: pg_restore dependencies
Previous Message Tom Lane 2009-04-10 21:16:14 Re: Re: [BUGS] BUG #4027: backslash escapingnotdisabled inplpgsql