Re: quoting psql varible as identifier
Pavel Stehule <pavel.stehule@gmail.com>
From: Pavel Stehule <pavel.stehule@gmail.com>
To: Robert Haas <robertmhaas@gmail.com>
Cc: Tom Lane <tgl@sss.pgh.pa.us>, Alvaro Herrera <alvherre@commandprompt.com>, PostgreSQL Hackers <pgsql-hackers@postgresql.org>
Date: 2010-01-11T04:22:25Z
Lists: pgsql-hackers
2010/1/11 Robert Haas <robertmhaas@gmail.com>: > On Tue, Jan 5, 2010 at 8:23 AM, Pavel Stehule <pavel.stehule@gmail.com> wrote: >> 2010/1/5 Tom Lane <tgl@sss.pgh.pa.us>: >>> Robert Haas <robertmhaas@gmail.com> writes: >>>> On Mon, Jan 4, 2010 at 2:51 PM, Pavel Stehule <pavel.stehule@gmail.com> wrote: >>>>> I don't have a problem to write second and safe fmtId >>>>> function (with technique used in dumputils don't need to modify >>>>> libpq), although fmtId do exactly what I need. I would to understand >>>>> to behave. >>> >>>> I think you mean that you would need to understand how it should >>>> behave - in which case I agree, but I think Tom spelled that out >>>> pretty clearly upthread: close PQescapeStringConn and adapt it to be >>>> PQescapeIdentifier. >>> >>> The more important point here is that fmtId doesn't do "exactly what you >>> need" in any case. fmtId is safe to use in pg_dump because pg_dump is >>> only expected to work with the same or older version of the backend. >>> It would not be safe to use it in libpq, which is expected to still work >>> with newer backends that might have more reserved words. >> >> So I finnaly moved to libpq PQescapeIdentConn function >> >> patch is attached. > > No longer applies, please rebase. ok I'll do it today Pavel > > Thanks, > > ...Robert >