Re: small explain patch

Поиск
Список
Период
Сортировка
От Liam Stewart
Тема Re: small explain patch
Дата
Msg-id 20020503102230.A2162@redhat.com
обсуждение исходный текст
Ответ на Re: small explain patch  (Tom Lane <tgl@sss.pgh.pa.us>)
Ответы Re: small explain patch  (Tom Lane <tgl@sss.pgh.pa.us>)
Список pgsql-patches
On Thu, May 02, 2002 at 05:42:13PM -0400, Tom Lane wrote:
> 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.)

Sure.

> How would you feel about quoting only if necessary
> (cf. quote_identifier)?

That would work nicely. I didn't put the call to quote_identifier inside
the stringStringInfo macro.

> > Aliases are explicitly labelled.
>
> I think that's just noise.  The explain output is verbose enough without
> adding noise words...

Hmm.. I kinda like having it there, but you are right in that it isn't
neccessary and doesn't add much. I'll live with it being out.

Liam

--
Liam Stewart :: Red Hat Canada, Ltd. :: liams@redhat.com

Вложения

В списке pgsql-patches по дате отправления:

Предыдущее
От: Tom Lane
Дата:
Сообщение: Re: small s_lock.c bug
Следующее
От: Tom Lane
Дата:
Сообщение: Re: small explain patch