Hi.
> Wow, I have no idea what that means. I am wondering if we should just
> remove this TODO item. We don't even support comments on indexed
> columns, so why would pg_dump need to dump it?
Oh I see.
But we still can comment on indexed columns like following on 8.3.7,
is it unsupported feature?
create table person (social_no integer, name text, age integer, uri
text, PRIMARY KEY (social_no));create index person_age on person using BTREE (age);comment on column person_age.age IS
'helloindex person_age.age';
And we can find the comment in pg_description table.
> Do we support comments on composite types
If we do, I will also write a patch for it.
Cheers.
-----
Taro Minowa(Higepon)
http://www.monaos.org/
http://code.google.com/p/mosh-scheme/
On Tue, Mar 24, 2009 at 11:10 PM, Bruce Momjian <bruce@momjian.us> wrote:
> higepon wrote:
>> Hi.
>> I found a TODO item "Add dumping of comments on index columns and
>> composite type columns" for pg_dump
>> and want to write a patch for it.
>>
>> But I'm not sure if I understand the problem correctly.
>> Does "Add dumping of comments on index columns" mean that
>> pg_dump should dump out COMMENT statements like following?
>>
>> COMMENT ON COLUMN some_index.index_column1 IS 'Hello column1';
>>
>> Can anybody give me some advice on this?
>
> Wow, I have no idea what that means. I am wondering if we should just
> remove this TODO item. We don't even support comments on indexed
> columns, so why would pg_dump need to dump it?
>
> The full text is:
>
> Add dumping of comments on index columns and composite type columns
>
> Do we support comments on composite types?
>
> This item first appeared on the TODO list in Postgres 8.0.
>
> --
> Bruce Momjian <bruce@momjian.us> http://momjian.us
> EnterpriseDB http://enterprisedb.com
>
> + If your life is a hard drive, Christ can be your backup. +
>