Hi,
On 17.12.2020 3:31, Zhihong Yu wrote:
Hi,
For EventTriggerOnConnect():
+ PG_CATCH();
+ {
...
+ AbortCurrentTransaction();
+ return;
Should runlist be freed in the catch block ?
No need: it is allocated in transaction memory context and removed on
transaction abort.
+ gettext_noop("In case of errors in the ON
client_connection EVENT TRIGGER procedure, this parameter can be used
to disable trigger activation and provide access to the database."),
I think the text should be on two lines (current line too long).
Thank you, fixed.
--
Konstantin Knizhnik
Postgres Professional: http://www.postgrespro.com
The Russian Postgres Company
