Обсуждение: Support event trigger for logoff

Hi, hackers

I see [1] has already implemented on login event trigger, why not implement
the logoff event trigger?

My friend Song Jinzhou and I try to implement the logoff event trigger, so
attach it.

Here is a problem with the regression test when using \c to create a new
session, because it might be running concurrently, which may lead to the
checking being unstable.

Any thoughts?

[1] https://www.postgresql.org/message-id/0d46d29f-4558-3af9-9c85-7774e14a7709%40postgrespro.ru

--
Regards,
Japin Li

Japin Li <japinli@hotmail.com> writes:
> I see [1] has already implemented on login event trigger, why not implement
> the logoff event trigger?

What happens if the session crashes, or otherwise fails unexpectedly?

> Here is a problem with the regression test when using \c to create a new
> session, because it might be running concurrently, which may lead to the
> checking being unstable.

> Any thoughts?

Let's not go there at all.  I don't think there's enough field
demand to justify dealing with this concept.

            regards, tom lane

On Sat, 20 Apr 2024 at 01:36, Tom Lane <tgl@sss.pgh.pa.us> wrote:
> Japin Li <japinli@hotmail.com> writes:
>> I see [1] has already implemented on login event trigger, why not implement
>> the logoff event trigger?
>
> What happens if the session crashes, or otherwise fails unexpectedly?
>

I am currently unsure how to handle such situations, but I believe it is not
only for logoff events.

>> Here is a problem with the regression test when using \c to create a new
>> session, because it might be running concurrently, which may lead to the
>> checking being unstable.
>
>> Any thoughts?
>
> Let's not go there at all.  I don't think there's enough field
> demand to justify dealing with this concept.
>

Thanks for your point out this for me.

--
Regards,
Japin Li