Re: On login trigger: take three
| От | Ted Yu |
|---|---|
| Тема | Re: On login trigger: take three |
| Дата | |
| Msg-id | CALte62wg7NC2cS+KoojuvK7VTn0qdY+O6Q2mTLgw8Oc-yab5BQ@mail.gmail.com обсуждение исходный текст |
| Ответ на | Re: On login trigger: take three (Mikhail Gribkov <youzhick@gmail.com>) |
| Ответы |
Re: On login trigger: take three
|
| Список | pgsql-hackers |
On Sat, Dec 17, 2022 at 3:46 AM Mikhail Gribkov <youzhick@gmail.com> wrote:
Hi Nikita,> Mikhail, I've checked the patch and previous discussion,> the condition mentioned earlier is still actual:Thanks for pointing this out! My bad, I forgot to fix the documentation example.Attached v34 has this issue fixed, as well as a couple other problems with the example code.--best regards,
Mikhail A. Gribkov
Hi,
bq. in to the system
'in to' -> into
bq. Any bugs in a trigger procedure
Any bugs -> Any bug
+ if (event == EVT_Login)
+ dbgtag = CMDTAG_LOGIN;
+ else
+ dbgtag = CreateCommandTag(parsetree);
+ dbgtag = CMDTAG_LOGIN;
+ else
+ dbgtag = CreateCommandTag(parsetree);
The same snippet appears more than once. It seems CMDTAG_LOGIN handling can be moved into `CreateCommandTag`.
Cheers
В списке pgsql-hackers по дате отправления: