Re: LIKE INCLUDING COMMENTS code is a flight of fancy

Поиск
Список
Период
Сортировка
От Tom Lane
Тема Re: LIKE INCLUDING COMMENTS code is a flight of fancy
Дата
Msg-id 29931.1261512855@sss.pgh.pa.us
обсуждение исходный текст
Ответ на Re: LIKE INCLUDING COMMENTS code is a flight of fancy  (Takahiro Itagaki <itagaki.takahiro@oss.ntt.co.jp>)
Ответы Re: LIKE INCLUDING COMMENTS code is a flight of fancy  (Andrew Dunstan <andrew@dunslane.net>)
Список pgsql-hackers
Takahiro Itagaki <itagaki.takahiro@oss.ntt.co.jp> writes:
> Tom Lane <tgl@sss.pgh.pa.us> wrote:
>> I suggest that we might want to just
>> rip out the support for copying comments on indexes.

> We have two related ToDo items below. They are a bit inconsintent,
> but they mean we should forbid COMMENT on columns of an index,
> or must have full-support of the feature.

> Which direction should we go?  As for me, forbidding comments on index
> columns seems to be a saner way because index can have arbitrary key
> names in some cases.

> - Forbid COMMENT on columns of an index
>     Postgres currently allows comments to be placed on the columns of
>     an index, but pg_dump doesn't handle them and the column names
>     themselves are implementation-dependent. 
>     http://archives.postgresql.org/message-id/27676.1237906577@sss.pgh.pa.us

> - pg_dump / pg_restore: Add dumping of comments on index columns and
>   composite type columns 
>     http://archives.postgresql.org/pgsql-hackers/2009-03/msg00931.php
>     (XXX: Comments on composite type columns can work now?)

I'm for forbidding comments on index columns.  The amount of work
required to support the feature fully seems far out of proportion to
its value.

In any case, if pg_dump drops such comments (which I had forgotten,
but it seems true after a quick look at the code), then we could
certainly get away with having LIKE not copy them.  That would fix
the immediate problem.
        regards, tom lane


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

Предыдущее
От: Alvaro Herrera
Дата:
Сообщение: Re: Segfault from PL/Perl Returning vstring
Следующее
От: Heikki Linnakangas
Дата:
Сообщение: Re: Tuplestore should remember the memory context it's created in