Weird...but correct?

Started by Christopher Kings-Lynneover 21 years ago3 messages
#1Christopher Kings-Lynne
chriskl@familyhealth.com.au

It's bizarre how you can comment on columns in composite types!

CREATE TYPE test (a int4, b int4);
COMMENT ON COLUMN test.a IS 'A column';

Seems harmless, but should we allow it?

Chris

#2Christopher Kings-Lynne
chriskl@familyhealth.com.au
In reply to: Christopher Kings-Lynne (#1)
Re: Weird...but correct?

CREATE TYPE test (a int4, b int4);
COMMENT ON COLUMN test.a IS 'A column';

Seems harmless, but should we allow it?

Actually, currently it's bad because such comments will not be dumped by
pg_dump. Shall I fix pg_dump?

Chris

#3Tom Lane
tgl@sss.pgh.pa.us
In reply to: Christopher Kings-Lynne (#1)
Re: Weird...but correct?

Christopher Kings-Lynne <chriskl@familyhealth.com.au> writes:

It's bizarre how you can comment on columns in composite types!
CREATE TYPE test (a int4, b int4);
COMMENT ON COLUMN test.a IS 'A column';

And not only that, but:

regression=# \d+ test
Composite type "public.test"
Column | Type | Description
--------+---------+-------------
a | integer | A column
b | integer |

Seems harmless, but should we allow it?

Why not?

Shall I fix pg_dump?

Yes please, if it doesn't seem outlandishly hard. (From memory it might
just be adding a subroutine call to dumpType...)

regards, tom lane