On Thu, Nov 11, 2021 at 8:56 PM Daniel Gustafsson <dan...@yesql.se> wrote: > > + This flag is used to avoid extra lookups on the pg_event_trigger > table during each backend startup. > This should be <structname>pg_event_trigger</structname>. Sorry, missed that > one at that last read-through. > Thanks, noted.
> > - dathaslogintriggers -> dathasloginevttriggers flag rename (too > > long?) > > I'm not crazy about this name, "evt" is commonly the abbreviation of "event > trigger" (used in evtcache.c etc) so "dathasloginevt" would IMO be better. > That being said, that's still not a very readable name, maybe someone else has > an even better suggestion? > Yes you're right, in this case I was wrongly treating "evt" as an abbreviation for "event". I agree "dathasloginevt" would be better. Regards, Greg Nancarrow Fujitsu Australia