CREATE TABLE LIKE INCLUDING CONSTRAINTS

Started by Bruce Momjianabout 20 years ago3 messagespatches
Jump to latest
#1Bruce Momjian
bruce@momjian.us

I just managed to crash the server so I guess this isn't finished, but I'm
posting it in the "post early post often" spirit. Aside from error checks it
also needs docs and tests of course.

This patch implements an option to copy check constraints when using LIKE.
Because the standard specifically excludes constraints (except NOT NULL) from
being copied it defaults to off and has to be explicitly requested by the user
using the nonstandard extension "INCLUDING CONSTRAINTS".

This is especially useful in combination with ALTER TABLE INHERIT since
without it there's no convenient way to create eligible tables for adding to
the inheritance tree. The user would have to manually reenter every check
constraint.

Question:

. Is nodeToString() the right thing here?

Currently only check constraints are added because only check constraints are
handled by inheritance. I intend to add foreign key constraints when I add
them to the rest of inheritance which will necessitate a scan of pg_constraint
instead of using the relcache :(

postgres=# create table z (i integer check (i=0));
CREATE TABLE
postgres=# create table zz (like z including constraints);
CREATE TABLE
postgres=# \d zz
Table "public.zz"
Column | Type | Modifiers
--------+---------+-----------
i | integer |
Check constraints:
"z_i_check" CHECK (i = 0)

Attachments:

like.patch2application/octet-streamDownload+102-39
#2Bruce Momjian
bruce@momjian.us
In reply to: Bruce Momjian (#1)
Re: CREATE TABLE LIKE INCLUDING CONSTRAINTS

Fixed the bug, added docs and tests.

Attachments:

like.patch3application/octet-streamDownload+147-47
#3Bruce Momjian
bruce@momjian.us
In reply to: Bruce Momjian (#2)
Re: CREATE TABLE LIKE INCLUDING CONSTRAINTS

Greg Stark <gsstark@MIT.EDU> writes:

This patch implements an option to copy check constraints when using LIKE.

Ah, found a problem. I need to do a change_varattnos_of_a_node() call here.
Should this function maybe be promoted to some other file like ruleutils.c?

--
greg