From: | Pavel Stehule <pavel(dot)stehule(at)gmail(dot)com> |
---|---|
To: | Konstantin Knizhnik <k(dot)knizhnik(at)postgrespro(dot)ru> |
Cc: | Greg Nancarrow <gregn4422(at)gmail(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: On login trigger: take three |
Date: | 2020-12-26 07:00:01 |
Message-ID: | CAFj8pRAsozLS0qz0DYutOt7aUbGs-=zWSKnMUZ+xK4A0NnfATA@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
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
Regards
Pavel
>
> Konstantin Knizhnik
> Postgres Professional: http://www.postgrespro.com
> The Russian Postgres Company
>
>
Attachment | Content-Type | Size |
---|---|---|
on_connect_event_trigger-13.patch | text/x-patch | 34.5 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Pavel Stehule | 2020-12-26 07:03:16 | Re: On login trigger: take three |
Previous Message | Bharath Rupireddy | 2020-12-26 06:27:43 | Re: Better client reporting for "immediate stop" shutdowns |