Re: FKey not enforced resulting in broken Dump/Reload

From: Robert Haas <robertmhaas(at)gmail(dot)com>
To: Rod Taylor <rod(dot)taylor(at)gmail(dot)com>
Cc: pgsql-hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: FKey not enforced resulting in broken Dump/Reload
Date: 2013-07-19 17:15:13
Message-ID: CA+TgmobjBgruNe2ptRFgLrx86Wc7R400rqkRqzJogyUP_CBXyA@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Fri, Jul 19, 2013 at 12:58 PM, Rod Taylor <rod(dot)taylor(at)gmail(dot)com> wrote:
> A poorly coded trigger on the referencing table has the ability to break
> foreign keys, and as a result create a database which cannot be dumped and
> reloaded.
>
> The BEFORE DELETE trigger accidentally does RETURN NEW, which suppresses the
> DELETE action by the foreign key trigger. This allows the record from the
> referenced table to be deleted and the record in the referencing table to
> remain in place.
>
> While I don't expect Pg to do what the coder meant, but it should throw an
> error and not leave foreign key'd data in an invalid state.

This is a known limitation of our foreign key machinery. It might
well be susceptible to improvement, but I wouldn't count on anyone
rewriting it in the near future.

--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Alvaro Herrera 2013-07-19 17:20:17 Re: [HACKERS] getting rid of SnapshotNow
Previous Message Robert Haas 2013-07-19 17:13:37 Re: [HACKERS] getting rid of SnapshotNow