On Thu, Jan 12, 2023 at 12:26 PM Daniel Gustafsson <dan...@yesql.se> wrote:
> > On 11 Jan 2023, at 17:38, vignesh C <vignes...@gmail.com> wrote: > > > > On Tue, 29 Nov 2022 at 18:16, Daniel Gustafsson <dan...@yesql.se> wrote: > >> > >>> On 3 Nov 2022, at 21:47, Daniel Gustafsson <dan...@yesql.se> wrote: > >> > >>> The patch adds a new GUC, ignore_event_trigger with two option values, > 'all' > >>> and 'none' (the login event patch had 'login' as well). > >> > >> The attached v2 fixes a small bug which caused testfailures the CFBot. > > > > The patch does not apply on top of HEAD as in [1], please post a rebased > patch: > > The attached rebased v3 fixes the conflict. > > -- > Daniel Gustafsson https://vmware.com/ > > Hi, `this GUC allows to temporarily suspending event triggers.` It would be better to mention the name of GUC in the description. Typo: suspending -> suspend w.r.t. guc `ignore_event_trigger`, since it is supposed to disable event triggers for a short period of time, is there mechanism to turn it off (IGNORE_EVENT_TRIGGER_ALL) automatically ? Cheers