Re: GUC for temporarily disabling event triggers

From: Robert Haas <robertmhaas(at)gmail(dot)com>
To: Michael Paquier <michael(at)paquier(dot)xyz>
Cc: Daniel Gustafsson <daniel(at)yesql(dot)se>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Justin Pryzby <pryzby(at)telsasoft(dot)com>, Mikhail Gribkov <youzhick(at)gmail(dot)com>, pgsql-hackers(at)lists(dot)postgresql(dot)org
Subject: Re: GUC for temporarily disabling event triggers
Date: 2023-09-07 19:02:48
Message-ID: CA+Tgmobiwm3VCBiwbG0OCGPyq0AETgEtXAmtNngj_1EPwEX2xg@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Thu, Sep 7, 2023 at 1:57 AM Michael Paquier <michael(at)paquier(dot)xyz> wrote:
> + GUC_NOT_IN_SAMPLE | GUC_DISALLOW_IN_FILE
>
> I am a bit surprised by these two additions. Setting this GUC at
> file-level can be useful, as is documenting it in the control file if
> it provides some control of how a statement behaves, no?

Yeah, I don't think these options should be used.

> + Allow temporarily disabling execution of event triggers in order to
> + troubleshoot and repair faulty event triggers. All event triggers will
> + be disabled by setting it to <literal>true</literal>. Setting the value
> + to <literal>false</literal> will not disable any event triggers, this
> + is the default value. Only superusers and users with the appropriate
> + <literal>SET</literal> privilege can change this setting.
>
> Event triggers are disabled if setting this GUC to false, while true,
> the default, allows event triggers. The values are reversed in this
> description.

Woops.

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

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Melanie Plageman 2023-09-07 19:09:54 Re: Eliminate redundant tuple visibility check in vacuum
Previous Message Robert Haas 2023-09-07 18:42:00 Re: A minor adjustment to get_cheapest_path_for_pathkeys