Re: GUC for temporarily disabling event triggers

Daniel Gustafsson <daniel@yesql.se>

From: Daniel Gustafsson <daniel@yesql.se>
To: Robert Haas <robertmhaas@gmail.com>
Cc: Justin Pryzby <pryzby@telsasoft.com>, Mikhail Gribkov <youzhick@gmail.com>, pgsql-hackers@lists.postgresql.org
Date: 2023-04-03T13:15:16Z
Lists: pgsql-hackers
> On 3 Apr 2023, at 15:09, Robert Haas <robertmhaas@gmail.com> wrote:
> 
> On Mon, Apr 3, 2023 at 8:46 AM Daniel Gustafsson <daniel@yesql.se> wrote:
>> All comments above addressed in the attached v5, thanks for review!
> 
> I continue to think it's odd that the sense of this is inverted as
> compared with row_security.

I'm not sure I follow.  Do you propose that the GUC enables classes of event
triggers, the default being "all" (or similar) and one would remove the type of
EVT for which debugging is needed?  That doesn't seem like a bad idea, just one
that hasn't come up in the discussion (and I didn't think about).

--
Daniel Gustafsson




Commits

  1. Add GUC for temporarily disabling event triggers

  2. Fix typos in code and comments