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
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 |
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 |