public inbox for [email protected]  

help / color / mirror / Atom feed

From: Tom Lane <[email protected]>
To: Thomas Munro <[email protected]>
Cc: Bruce Momjian <[email protected]>
Cc: [email protected]
Cc: [email protected]
Cc: Andrew Gierth <[email protected]>
Subject: Re: Trigger behaviour not as stated
Date: Sun, 28 Jan 2018 14:18:38 -0500
Message-ID: <[email protected]> (raw)
In-Reply-To: <CAEepm=1cAO8y4dFv_xd=43iP7f4T3A2hYAzjgjQagHtt7H87Fw@mail.gmail.com>
References: <[email protected]>
	<[email protected]>
	<[email protected]>
	<CAEepm=1cAO8y4dFv_xd=43iP7f4T3A2hYAzjgjQagHtt7H87Fw@mail.gmail.com>

Thomas Munro <[email protected]> writes:
> On Thu, Jan 25, 2018 at 11:07 AM, Bruce Momjian <[email protected]> wrote:
>> ... The attached patch updates the docs to say
>> statement-level triggers fire on the "referenced" table, while row-level
>> triggers fire only on the "affected" table, (vs. all affected tables)
>> even if they are not referenced in the query.  I would backpatch this to
>> PG 10.

> I was trying to convey that, but it does seem a little terse and
> cryptic.  Your addition of "referenced" and "only" make it clearer.

Hm, the first part of Bruce's change seems fine, but I think this wording:

     ...  In contrast,
!    row-level triggers are fired only on affected partitions or child tables,
!    even if they are not referenced in the query.

is still confusing.  How about something like

     In contrast, row-level triggers are fired for each actual row change,
     including changes in partitions or child tables that are not directly
     named in the query.

Possibly "row operation" would be better than "row change".

			regards, tom lane

view thread (13+ messages)  latest in thread

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: [email protected]
  Cc: [email protected], [email protected], [email protected], [email protected], [email protected]
  Subject: Re: Trigger behaviour not as stated
  In-Reply-To: <[email protected]>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox