From: | "Andrew Dunstan" <andrew(at)dunslane(dot)net> |
---|---|
To: | <pgman(at)candle(dot)pha(dot)pa(dot)us> |
Cc: | <tgl(at)sss(dot)pgh(dot)pa(dot)us>, <andrew(at)dunslane(dot)net>, <pg(at)rbt(dot)ca>, <pgsql-hackers(at)postgresql(dot)org>, <chriskl(at)familyhealth(dot)com(dot)au> |
Subject: | Re: [PATCHES] Escape handling in strings |
Date: | 2005-06-17 01:03:04 |
Message-ID: | 1293.24.211.165.134.1118970184.squirrel@www.dunslane.net |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers pgsql-patches |
Bruce Momjian said:
> OK, the current patch warns about two things, \' with one message, and
> any backslash in a non-E string with a different message. The \'
> message can easily be avoided in clients even in 8.0 by using '', but
> for E'', there is no way to prepare an application before upgrading to
> 8.1 because 8.0 doesn't have E''. (We can add E'' in a subrelease, but
> what percentage of users are going to upgrade to that?) This is why I
> think we need to add a GUC to allow the warning to be turned off. To
> be clear, the GUC is to control the warning, not the query behavior.
>
> We could go with the second warning only in 8.2, but that seems too
> confusing --- we should deal with the escape issue in two stages,
> rather than three.
>
So you don't agree with Tom's suggestion to implement E'' a full cycle
before removing backslash processing in standard strings? Or have I
misunderstood again?
cheers
andrew
From | Date | Subject | |
---|---|---|---|
Next Message | Tim Allen | 2005-06-17 01:04:33 | Re: Autovacuum in the backend |
Previous Message | Josh Berkus | 2005-06-17 00:31:37 | DTrace Probes? |
From | Date | Subject | |
---|---|---|---|
Next Message | Bruce Momjian | 2005-06-17 01:41:05 | Re: [PATCHES] Escape handling in strings |
Previous Message | Bruce Momjian | 2005-06-16 17:53:59 | Re: fsync under win32 |