Re: That dump-comments-on-composite-type-columns patch... - Mailing list pgsql-patches

From Christopher Kings-Lynne
Subject Re: That dump-comments-on-composite-type-columns patch...
Date
Msg-id 4115B1BF.6020509@familyhealth.com.au
Whole thread Raw
In response to That dump-comments-on-composite-type-columns patch...  (Tom Lane <tgl@sss.pgh.pa.us>)
Responses Re: That dump-comments-on-composite-type-columns patch...
List pgsql-patches
> $ pg_dump regression >zzz.out
> pg_dump: SQL command failed
> pg_dump: Error message from server: ERROR:  "complex" is a composite type
> pg_dump: The command was: COPY public.complex (r, i) TO stdout;
> $

That could be fixed by just checking the relkind when dumping table
data, but hey.

> I suspect it had more subtle problems too, because dumpTableComments
> would have attached the comments to the dumpid associated with the
> TableInfo entry, which isn't the object that will get dumped.  So it
> seems moderately likely that there would have been a potential for
> misordering of the output.

Ok.

> I think it's probably a fundamentally bad idea to be putting composite
> types into pg_dump's TableInfo array, because they just really aren't
> tables at all.  If you want to try again, I'd suggest writing a variant
> of dumpTableComment that takes a TypeInfo and the attribute-names query
> data obtained by dumpCompositeType.

You mean unlike views, sequences and all other kinds of junk? :)

OK, I can do this, but I don't think I'll have time for the first beta.

Chris

ps. Did you back out the moving of owner to commands as well?

pgsql-patches by date:

Previous
From: Bruce Momjian
Date:
Subject: Re: Win32 tablespace
Next
From: markir@coretech.co.nz
Date:
Subject: Re: Win32 tablespace