From: | Nathan Long <him(at)nathanmlong(dot)com> |
---|---|
To: | Laurenz Albe <laurenz(dot)albe(at)cybertec(dot)at> |
Cc: | pgsql-docs(at)lists(dot)postgresql(dot)org |
Subject: | Re: Clarify how triggers relate to transactions |
Date: | 2021-04-28 17:18:40 |
Message-ID: | CAOyE-6ZsrpJT2HRBRzGoixZDvRUYZnrvgRXFfZYs1WRW+rpZKQ@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-docs pgsql-hackers |
Gotcha. Where would I go to make the PR?
On Wed, Apr 28, 2021, 7:24 AM Laurenz Albe <laurenz(dot)albe(at)cybertec(dot)at> wrote:
> On Tue, 2021-04-27 at 14:26 +0000, PG Doc comments form wrote:
> > The following documentation comment has been logged on the website:
> >
> > Page: https://www.postgresql.org/docs/13/sql-createtrigger.html
> > Description:
> >
> > https://www.postgresql.org/docs/current/sql-createtrigger.html mentions
> the
> > word "transaction" only once, in reference specifically to constraint
> > triggers: "They can be fired either at the end of the statement causing
> the
> > triggering event, or at the end of the containing transaction; in the
> latter
> > case they are said to be deferred."
> >
> > If I understand correctly, it would be helpful to add this sentence or a
> > corrected version of it: "Triggers always execute in the same
> transaction as
> > the triggering event, and if a trigger fails, the transaction is rolled
> > back."
>
> Good idea in principle, but I'd put that information on
> https://www.postgresql.org/docs/current/trigger-definition.html
>
> Yours,
> Laurenz Albe
>
>
From | Date | Subject | |
---|---|---|---|
Next Message | PG Doc comments form | 2021-04-28 18:16:07 | Wrong type for the left argument of the operator "|/" and "||/" in chapter 9.3 |
Previous Message | Laurenz Albe | 2021-04-28 11:24:53 | Re: Clarify how triggers relate to transactions |
From | Date | Subject | |
---|---|---|---|
Next Message | Laurenz Albe | 2021-04-28 18:17:49 | Re: Clarify how triggers relate to transactions |
Previous Message | Tom Lane | 2021-04-28 16:45:35 | Re: WIP: WAL prefetch (another approach) |