From: | NikhilS <nikkhils(at)gmail(dot)com> |
---|---|
To: | "Tom Lane" <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | "Bruce Momjian" <bruce(at)momjian(dot)us>, "Chris Fischer" <Chris(dot)Fischer(at)channeladvisor(dot)com>, pgsql-bugs(at)postgresql(dot)org |
Subject: | Re: Problem identifying constraints which should not be inherited |
Date: | 2008-03-20 06:49:49 |
Message-ID: | d3c4af540803192349n1c022feemdc8bba68956d56b6@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-bugs pgsql-hackers |
Hi,
On Wed, Mar 19, 2008 at 8:23 PM, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
> NikhilS <nikkhils(at)gmail(dot)com> writes:
> > On Fri, Mar 7, 2008 at 6:37 AM, Bruce Momjian <bruce(at)momjian(dot)us> wrote:
> >> Added to TODO:
> >> o Require all CHECK constraints to be inherited
>
> > PFA, a small patch attached which should fix this.
>
> If it's a small patch, it's wrong by definition. AFAICS there is no way
> to fix this correctly that doesn't involve catalog changes. The point
> of the TODO is that you have to enforce that the inherited constraint
> sticks around, eg can't be dropped on a child table while it's still
> present on the parent. There are implications for pg_dump too.
>
Ok, I understand. But even then this could patch could be considered even if
it does not solve the TODO completely, no? It atleast disallows ONLY ADD
CONSTRAINT on the parent.
Regards,
Nikhils
--
EnterpriseDB http://www.enterprisedb.com
From | Date | Subject | |
---|---|---|---|
Next Message | Heikki Linnakangas | 2008-03-20 11:18:44 | Re: 8.3 can't convert cyrillic text from 'iso-8859-5' to other cyrillic 8-bit encoding |
Previous Message | Sergey Burladyan | 2008-03-20 03:33:03 | Re: 8.3 can't convert cyrillic text from 'iso-8859-5' to other cyrillic 8-bit encoding |
From | Date | Subject | |
---|---|---|---|
Next Message | Pavan Deolasee | 2008-03-20 07:00:42 | Re: count(*) performance improvement ideas |
Previous Message | Tatsuo Ishii | 2008-03-20 06:04:15 | Re: Proposal: new large object API |