From: | David Fetter <david(at)fetter(dot)org> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | PG Hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Arrays of Complex Types |
Date: | 2007-03-06 21:05:56 |
Message-ID: | 20070306210556.GA893@fetter.org |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers pgsql-patches |
On Fri, Mar 02, 2007 at 06:59:50PM -0500, Tom Lane wrote:
> David Fetter <david(at)fetter(dot)org> writes:
> > 1. In src/backend/commands/tablecmds.c, change DefineRelation as
> > follows:
> > * After the first call to heap_create_with_catalog, construct and
> > do another call to for the array type.
>
> I'm still not happy about the idea of doing this for every relation
> (and doing it for sequences and indexes would be the height of
> wastefulness). How about we only do it for composite types?
How about doing it for user-defined tables, views and composite types,
and skipping ?
> > * Add an appropriate pg_depend entry.
> > 2. Change RemoveRelation to reflect the above.
>
> You only need one of those two: either you drop by hand or you let the
> dependency machinery deal with it. Not both.
pg_depend it is, then :)
> > Does the above make sense? Have I missed anything critical?
>
> Ummm ... making it actually work? Possibly that just falls out, but
> I'm not sure.
>
> If it turns out that it does Just Work, you might take a stab at
> arrays of domains too.
OK.
I noticed something in src/backend/commands/tablecmds.c which worries
me, namely that it ignores functions and views. It should at least be
checking that the typeoid isn't in pg_proc.prorettype or
pg_proc.proargtypes, and if possible, the DECLARE section of pl/pgsql
functions.
Is there a way to do SQL at that place in the back-end, or is there
some different kind of Magick(TM) needed to access these kinds of
things at that level?
Cheers,
D
--
David Fetter <david(at)fetter(dot)org> http://fetter.org/
phone: +1 415 235 3778 AIM: dfetter666
Skype: davidfetter
Remember to vote!
Consider donating to PostgreSQL: http://www.postgresql.org/about/donate
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2007-03-06 21:14:07 | Re: Arrays of Complex Types |
Previous Message | Andrew Dunstan | 2007-03-06 20:34:53 | Re: Auto creation of Partitions |
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2007-03-06 21:14:07 | Re: Arrays of Complex Types |
Previous Message | Andrew Dunstan | 2007-03-06 20:34:53 | Re: Auto creation of Partitions |