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 обсуждение исходный текст |
| Ответ на | 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
|
| Список | pgsql-hackers |
Tom Lane wrote: > Brendan Jurd <direvus@gmail.com> 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_dumpdoesn't handle them and the column names themselvesare implementation-dependent. -- Bruce Momjian <bruce@momjian.us> http://momjian.us EnterpriseDB http://enterprisedb.com + If your life is a hard drive, Christ can be your backup. +
В списке pgsql-hackers по дате отправления: