From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | Liam Stewart <liams(at)redhat(dot)com> |
Cc: | PostgreSQL Patches List <pgsql-patches(at)postgresql(dot)org> |
Subject: | Re: small explain patch |
Date: | 2002-05-02 21:42:13 |
Message-ID: | 18400.1020375733@sss.pgh.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-patches |
Liam Stewart <liams(at)redhat(dot)com> writes:
> Attached is a small patch that clarifies EXPLAIN output a little bit.
> Output can be a bit confusing if there are spaces in table and index
> names and aliases so the names in question are surrounded by ' '.
If we do this, we should use double quotes "...". A double-quoted
string is an identifier, a single-quoted string is a literal; and
it doesn't help anyone for the system's messages to get it backwards.
(I've been intending for awhile to fix all the elog messages that get
this wrong.)
How would you feel about quoting only if necessary
(cf. quote_identifier)?
> Aliases are explicitly labelled.
I think that's just noise. The explain output is verbose enough without
adding noise words...
regards, tom lane
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2002-05-02 21:45:13 | Re: small s_lock.c bug |
Previous Message | Liam Stewart | 2002-05-02 18:14:18 | small explain patch |