so 26. 12. 2020 v 8:00 odesÃlatel Pavel Stehule <pavel.steh...@gmail.com> napsal:
> Hi > > > Thank you. >> I have applied all your fixes in on_connect_event_trigger-12.patch. >> >> Concerning enable_client_connection_trigger GUC, I think that it is >> really useful: it is the fastest and simplest way to disable login triggers >> in case >> of some problems with them (not only for superuser itself, but for all >> users). Yes, it can be also done using "ALTER EVENT TRIGGER DISABLE". >> But assume that you have a lot of databases with different login policies >> enforced by on-login event triggers. And you want temporary disable them >> all, for example for testing purposes. >> In this case GUC is most convenient way to do it. >> >> > There was typo in patch > > diff --git a/doc/src/sgml/config.sgml b/doc/src/sgml/config.sgml > index f810789..8861f1b 100644 > --- a/doc/src/sgml/config.sgml > +++ b/doc/src/sgml/config.sgml > @@ -1,4 +1,4 @@ > -<!-- doc/src/sgml/config.sgml --> > +\<!-- doc/src/sgml/config.sgml --> > > I have not any objections against functionality or design. I tested the > performance, and there are no negative impacts when this feature is not > used. There is significant overhead related to plpgsql runtime > initialization, but when this trigger will be used, then probably some > other PLpgSQL procedures and functions will be used too, and then this > overhead can be ignored. > > * make without warnings > * make check-world passed > * doc build passed > > Possible ToDo: > > The documentation can contain a note so usage connect triggers in > environments with short life sessions and very short fast queries without > usage PLpgSQL functions or procedures can have negative impact on > performance due overhead of initialization of PLpgSQL engine. > > I'll mark this patch as ready for committers > looks so this patch has not entry in commitfestapp 2021-01 Regards Pavel > Regards > > Pavel > > > >> >> Konstantin Knizhnik >> Postgres Professional: http://www.postgrespro.com >> The Russian Postgres Company >> >>