From: Tom Lane <tgl@sss.pgh.pa.us>
To: Thomas Munro <thomas.munro@enterprisedb.com>
cc: Bruce Momjian <bruce@momjian.us>, ian@thepathcentral.com,
        pgsql-docs@postgresql.org, Andrew Gierth <andrew@tao11.riddles.org.uk>
Subject: Re: Trigger behaviour not as stated
In-reply-to: 
 <CAEepm=1cAO8y4dFv_xd=43iP7f4T3A2hYAzjgjQagHtt7H87Fw@mail.gmail.com>
References: <20171129193934.27108.30796@wrigleys.postgresql.org>
 <20180124181008.GK17109@momjian.us> <20180124220701.GP17109@momjian.us>
 <CAEepm=1cAO8y4dFv_xd=43iP7f4T3A2hYAzjgjQagHtt7H87Fw@mail.gmail.com>
Comments: In-reply-to Thomas Munro <thomas.munro@enterprisedb.com>
	message dated "Sun, 28 Jan 2018 23:19:47 +1300"
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-ID: <28237.1517167118.1@sss.pgh.pa.us>
Date: Sun, 28 Jan 2018 14:18:38 -0500
Message-ID: <28238.1517167118@sss.pgh.pa.us>
Precedence: bulk

Thomas Munro <thomas.munro@enterprisedb.com> writes:
> On Thu, Jan 25, 2018 at 11:07 AM, Bruce Momjian <bruce@momjian.us> 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