Somewhat happier alter table add <check table constraint>

Started by Stephan Szaboabout 26 years ago2 messagespatches
Jump to latest
#1Stephan Szabo
sszabo@megazone23.bigpanda.com

This should be a slighly more complete patch for commands/command.c
AlterTableAddConstraint. The major changes from the last patch
are that it should hopefully check for references to temp tables
(not in the shadow case, but at defination time) from permanent tables in
foreign keys and refuse them and that it doesn't allow the table(s)
being constrained to be views (because those cases don't currently
work).

Stephan Szabo
sszabo@bigpanda.com

Attachments:

atac.patchtext/plain; charset=US-ASCII; name=atac.patchDownload+187-3
#2Bruce Momjian
bruce@momjian.us
In reply to: Stephan Szabo (#1)
Re: Somewhat happier alter table add <check table constraint>

Would you resumit this as a context diff, diff -c. Thanks. I will
apply it as soon as I receive it.

This should be a slighly more complete patch for commands/command.c
AlterTableAddConstraint. The major changes from the last patch
are that it should hopefully check for references to temp tables
(not in the shadow case, but at defination time) from permanent tables in
foreign keys and refuse them and that it doesn't allow the table(s)
being constrained to be views (because those cases don't currently
work).

Stephan Szabo
sszabo@bigpanda.com

Content-Description:

[ Attachment, skipping... ]

-- 
  Bruce Momjian                        |  http://candle.pha.pa.us
  pgman@candle.pha.pa.us               |  (610) 853-3000
  +  If your life is a hard drive,     |  830 Blythe Avenue
  +  Christ can be your backup.        |  Drexel Hill, Pennsylvania 19026