Re: GUC for temporarily disabling event triggers - Mailing list pgsql-hackers

From Robert Haas
Subject Re: GUC for temporarily disabling event triggers
Date
Msg-id CA+Tgmob38GJCyUe1eDrR_ygyrgJivgwzJpgzrAZ5AtPM2t3cig@mail.gmail.com
Whole thread Raw
In response to Re: GUC for temporarily disabling event triggers  (Daniel Gustafsson <daniel@yesql.se>)
Responses Re: GUC for temporarily disabling event triggers
List pgsql-hackers
On Wed, Sep 6, 2023 at 4:50 AM Daniel Gustafsson <daniel@yesql.se> wrote:
> > On 5 Sep 2023, at 17:29, Robert Haas <robertmhaas@gmail.com> wrote:
> > On Tue, Sep 5, 2023 at 8:12 AM Daniel Gustafsson <daniel@yesql.se> wrote:
>
> >> The attached version of the patch replaces it with a boolean flag for turning
> >> off all event triggers, and I also renamed it to the debug_xxx "GUC namespace"
> >> which seemed more appropriate.
> >
> > I don't care for the debug_xxx renaming, myself. I think that should
> > be reserved for things where we believe there's no reason to ever use
> > it in production/real life, or for things whose purpose is to emit
> > debugging messages. Neither is the case here.
>
> Fair enough, how about disable_event_trigger instead as per the attached?

I usually prefer to give things a positive sense, talking about
whether things are enabled rather than disabled. I'd do event_triggers
= off | on, like we have for row_security. YMMV, though.

--
Robert Haas
EDB: http://www.enterprisedb.com



pgsql-hackers by date:

Previous
From: Robert Haas
Date:
Subject: Re: generic plans and "initial" pruning
Next
From: Robert Haas
Date:
Subject: Re: Eager page freeze criteria clarification