From: | Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | Thomas Lockhart <thomas(at)fourpalms(dot)org>, PostgreSQL Hackers List <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: quotes in SET grammar |
Date: | 2002-02-26 17:56:50 |
Message-ID: | 200202261756.g1QHuoN16038@candle.pha.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Tom Lane wrote:
> Thomas Lockhart <thomas(at)fourpalms(dot)org> writes:
> > Possible cases look like
>
> > SET TIME ZONE 'pst8pdt';
> > SET TIME ZONE "pst8pdt";
> > SET DATESTYLE = "US","ISO";
>
> > Is there any objection to allowing both single- and double-quoted
> > strings in SET? Or should I remove the double-quoted variety altogether?
>
> I think it would be best to disallow the double-quoted form. If we
> allow it, then we will have a backwards-compatibility problem should
> we ever want to generalize SET to accept an expression (because
> double-quoted things are identifiers, not literals).
>
> However, I'm not sure *how* to disallow it without also disallowing
> unquoted words (since ultimately the productions reduce to ColId,
> and the lexer output doesn't distinguish quoted and unquoted
> identifiers). I don't think I want to go back to writing
> set whatever to 'on';
> so I guess I'll have to just grin and bear it.
>
> I agree that all the forms of SET should be consistent about what
> kinds of quoted or unquoted words they will take.
I see, because we allow non-quoted values, the quotes are accepted too.
Makes sense.
--
Bruce Momjian | http://candle.pha.pa.us
pgman(at)candle(dot)pha(dot)pa(dot)us | (610) 853-3000
+ If your life is a hard drive, | 830 Blythe Avenue
+ Christ can be your backup. | Drexel Hill, Pennsylvania 19026
From | Date | Subject | |
---|---|---|---|
Next Message | Bruce Momjian | 2002-02-26 18:05:37 | Re: COPY FROM is not 8bit clean |
Previous Message | Bruce Momjian | 2002-02-26 17:45:52 | Re: Implementation Proposal For Add Free Behind Capability |