Re: pg_dump Add dumping of comments on index columns

Поиск
Список
Период
Сортировка
Искать
От
Bruce Momjian
Тема
Re: pg_dump Add dumping of comments on index columns
Дата
Msg-id
200908080400.n7840Mt11264@momjian.us
Ответ на
Список
Дерево обсуждения
pg_dump Add dumping of comments on index columns higepon <higepon@gmail.com>
Re: pg_dump Add dumping of comments on index columns Bruce Momjian <bruce@momjian.us>
Re: pg_dump Add dumping of comments on index columns Tom Lane <tgl@sss.pgh.pa.us>
Re: pg_dump Add dumping of comments on index columns higepon <higepon@gmail.com>
Re: pg_dump Add dumping of comments on index columns Jaime Casanova <jcasanov@systemguards.com.ec>
Re: pg_dump Add dumping of comments on index columns Brendan Jurd <direvus@gmail.com>
Re: pg_dump Add dumping of comments on index columns Tom Lane <tgl@sss.pgh.pa.us>
Re: pg_dump Add dumping of comments on index columns Bruce Momjian <bruce@momjian.us>
Re: pg_dump Add dumping of comments on index columns Brendan Jurd <direvus@gmail.com>
Re: pg_dump Add dumping of comments on index columns higepon <higepon@gmail.com>
Re: pg_dump Add dumping of comments on index columns Robert Haas <robertmhaas@gmail.com>
Re: pg_dump Add dumping of comments on index columns higepon <higepon@gmail.com>
Re: pg_dump Add dumping of comments on index columns higepon <higepon@gmail.com>
Tom Lane wrote:
> Brendan Jurd  writes:
> > I've also done an initial review of the patch.  Everything looks sane
> > and the patch works as advertised.  I made a couple of minor tweaks
> > for code-style and comment consistency, and my version 3 is attached.
> 
> > I'm marking this patch Ready for Committer.
> 
> Applied with minor revisions --- mostly, it leaked memory in the case
> of no comments, and the query wasn't very schema-safe.

Just to verify, this patch was about comments on composite columns, not
about dumping comments on index columns (as the subject states), right? 
We do have a TODO for index column comments:
Forbid COMMENT on columns of an index    Postgres currently allows comments to be placed on the columns of anindex, but pg_dump doesn't handle them and the column names themselvesare implementation-dependent. 

--  Bruce Momjian          http://momjian.us EnterpriseDB                             http://enterprisedb.com
 + If your life is a hard drive, Christ can be your backup. +

В списке pgsql-hackers по дате отправления
От: Bruce Momjian
Дата:
От: Brendan Jurd
Дата:
FAQ