> On 7 Sep 2023, at 21:02, Robert Haas <robertmh...@gmail.com> wrote: > > On Thu, Sep 7, 2023 at 1:57 AM Michael Paquier <mich...@paquier.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.
Removed. >> + 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. Fixed. Since the main driver for this is to reduce the usage/need for single-user mode I also reworded the patch slightly. Instead of phrasing this as an alternative to single-user mode, I reversed it such that single-user mode is an alternative to this GUC. -- Daniel Gustafsson
v9-0001-Add-GUC-for-temporarily-disabling-event-triggers.patch
Description: Binary data