Re: Does 'instead of delete' trigger support modification of OLD

From: Eugen Konkov <kes-kes(at)yandex(dot)ru>
To: Bruce Momjian <bruce(at)momjian(dot)us>
Cc: PostgreSQL-development <pgsql-hackers(at)postgreSQL(dot)org>, pgsql-docs(at)lists(dot)postgresql(dot)org
Subject: Re: Does 'instead of delete' trigger support modification of OLD
Date: 2019-11-07 09:20:32
Message-ID: 1771593631.20191107112032@yandex.ru
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-docs pgsql-hackers

> I looked in the CREATE TRIGGER manual page and found this:

> https://www.postgresql.org/docs/12/sql-createtrigger.html
> If the trigger fires before or instead of the event, the trigger
> can skip the operation for the current row, or change the row
> being inserted (for INSERT and UPDATE operations only).

> I don't see the "(for INSERT and UPDATE operations only)" language in
> the main trigger documentation,
> https://www.postgresql.org/docs/current/trigger-definition.html. I have
> written the attached patch to fix that. Does that help?

No. If we document that PG does not allow to modify OLD at instead
of trigger, the we can not implement that. Probably we can put note
that "currently modification of the trigger row for RETURNING is not
implemented"

> As far as allowing DELETE to modify the trigger row for RETURNING, I am
> not sure how much work it would take to allow that, but it seems like it
> is a valid requite, and if so, I can add it to the TODO list.

Yes, Add please into TODO the feature to "allowing DELETE to modify the trigger row
for RETURNING". Becuase, as I have described at first letter, without
this the RETURNING rows **does not correspond actually deleted data**

Thank you.

--
Best regards,
Eugen Konkov

In response to

Responses

Browse pgsql-docs by date

  From Date Subject
Next Message Eugen Konkov 2019-11-07 09:24:29 Re: Does 'instead of delete' trigger support modification of OLD
Previous Message Bruce Momjian 2019-11-07 03:42:30 Re: auto_explain.log_min_duration is limit for nested statement

Browse pgsql-hackers by date

  From Date Subject
Next Message Grigory Smolkin 2019-11-07 09:22:28 Re: [proposal] recovery_target "latest"
Previous Message Julien Rouhaud 2019-11-07 09:20:17 Re: Collation versioning