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 по дате отправления