Re: Fix disabled triggers with deferred constraints

From: Stephan Szabo <sszabo(at)megazone23(dot)bigpanda(dot)com>
To: Neil Conway <nconway(at)klamath(dot)dyndns(dot)org>
Cc: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us>, Elliot Lee <sopwith(at)redhat(dot)com>, <pgsql-patches(at)postgresql(dot)org>
Subject: Re: Fix disabled triggers with deferred constraints
Date: 2002-08-12 22:22:08
Message-ID: 20020812152141.H55884-100000@megazone23.bigpanda.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-patches


On Mon, 12 Aug 2002, Stephan Szabo wrote:

> On 7 Aug 2002, Neil Conway wrote:
>
> > Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> writes:
> > > Neil Conway <nconway(at)klamath(dot)dyndns(dot)org> writes:
> > > > Elliot Lee <sopwith(at)redhat(dot)com> writes:
> > > > I remember looking at this issue and not doing anything because I
> > > > couldn't decide whether the test for enabled status should occur when
> > > > the trigger is queued or when it is executed --- or, perhaps, both?
> > > > Is there anything in the standard about it?
> >
> > [...]
> >
> > > I think we're still waiting for someone to figure out what the behavior
> > > should be per spec.
> >
> > I took a brief look at SQL99, but I couldn't find anything regarding
> > this issue (AFAICS it doesn't mention "disabled triggers" at all). But
> > given my prior track record for divining information from the
> > standards, perhaps someone should double-check :-)
> >
> > I did notice some behavior which we don't implement AFAIK:
> >
> > If the constraint mode is /deferred/, then the constraint is
> > effectively checked when the constraint mode is changed to
> > /immediate/ either explicitely by execution of a <set
> > constraints mode statement>, or implicitely at the end of the
> > current SQL-transaction.
> >
> > (SQL99, Section 4.17.1, paragraph 3)
> >
> > We don't recheck any outstanding deferred constraints when the
> > constraint mode is explicitly switched back to IMMEDIATE, as the
> > standard says we should.
>
> (have been out since last wed, responding now)
> Seems to work for me in some cases if I understand correctly.

Replying to myself. This is before doing a cvs update for the patch
you sent.

In response to

Browse pgsql-patches by date

  From Date Subject
Next Message Tom Lane 2002-08-12 22:25:25 Re: Antw: Re: Patch for NetWare support
Previous Message Stephan Szabo 2002-08-12 22:21:04 Re: Fix disabled triggers with deferred constraints