Re: Arrays of Complex Types

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: David Fetter <david(at)fetter(dot)org>
Cc: PG Hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Arrays of Complex Types
Date: 2007-03-02 23:59:50
Message-ID: 22332.1172879990@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers pgsql-patches

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?

> * 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.

> 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.

regards, tom lane

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Jeff Davis 2007-03-03 00:25:18 Re: Synchronized Scan update
Previous Message Josh Berkus 2007-03-02 23:49:53 Re: Synchronized Scan update

Browse pgsql-patches by date

  From Date Subject
Next Message Andrew Dunstan 2007-03-03 00:42:14 Re: Arrays of Complex Types
Previous Message David Fetter 2007-03-02 23:40:16 Arrays of Complex Types