Re: Doc patch to note which system catalogs have oids

Поиск
Список
Период
Сортировка
От Stephen Frost
Тема Re: Doc patch to note which system catalogs have oids
Дата
Msg-id 20120925011800.GJ1267@tamriel.snowman.net
обсуждение исходный текст
Ответ на Re: Doc patch to note which system catalogs have oids  (Tom Lane <tgl@sss.pgh.pa.us>)
Ответы Re: Doc patch to note which system catalogs have oids  ("Karl O. Pinc" <kop@meme.com>)
Re: Doc patch to note which system catalogs have oids  (Robert Haas <robertmhaas@gmail.com>)
Список pgsql-hackers
Tom,

* Tom Lane (tgl@sss.pgh.pa.us) wrote:
> I think this is fundamentally wrong, or at least misleading, because it
> documents OID as if it were an ordinary column.  Somebody who did
> "select * from pg_class" and didn't see any "oid" in the result would
> think the docs were wrong.

Given that it's been quite some time since we defaulted to including
OIDs in tables, and the high level of confusion that individuals trying
to join pg_class and pg_namespace together go through due to select *
not including the oid column, I wonder if perhaps we should consider
changing that.  Might be possible to do for just the catalog tables (to
minimize the risk of breaking poorly-written applications), or provide
a GUC to control including the oid column in select *.

> It's possible that it's worth expending a boilerplate paragraph in each
> of those pages to say "this catalog has OIDs" (or that it doesn't).
> But this isn't the way.

I'm afraid I disagree with this.  The oid column, in the system
catalog, is user-facing and I like having it included as a column in the
table in the docs, so users know what to use when doing joins.
Including something in the boilerplate about it not being shown by
default (or in the description in the table) might be alright, if we
don't change that.
Thanks,
    Stephen

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

Предыдущее
От: Rural Hunter
Дата:
Сообщение: Re: [ADMIN] pg_upgrade from 9.1.3 to 9.2 failed
Следующее
От: Tom Lane
Дата:
Сообщение: Re: Oid registry