Support event trigger for logoff

Started by Japin Lialmost 2 years ago3 messageshackers
Jump to latest
#1Japin Li
japinli@hotmail.com

Hi, hackers

I see [1]/messages/by-id/0d46d29f-4558-3af9-9c85-7774e14a7709@postgrespro.ru 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]: /messages/by-id/0d46d29f-4558-3af9-9c85-7774e14a7709@postgrespro.ru

--
Regards,
Japin Li

Attachments:

v1-0001-Add-support-event-triggers-for-logoff.patchtext/x-diffDownload+309-10
#2Tom Lane
tgl@sss.pgh.pa.us
In reply to: Japin Li (#1)
Re: Support event trigger for logoff

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

#3Japin Li
japinli@hotmail.com
In reply to: Tom Lane (#2)
Re: Support event trigger for logoff

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