| 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:24:29 | 
| Message-ID: | 352093181.20191107112429@yandex.ru | 
| Views: | Whole Thread | Raw Message | Download mbox | Resend email | 
| Thread: | |
| Lists: | pgsql-docs pgsql-hackers | 
Hello Eugen,
Thursday, November 7, 2019, 11:20:32 AM, you wrote:
>> 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"
sorry, typo. Please read:
"currently  modification of the trigger row for DELETE RETURNING is notimplemented"
>> 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
| From | Date | Subject | |
|---|---|---|---|
| Next Message | PG Doc comments form | 2019-11-07 14:17:58 | Nit: "Immutable" should be "pure" | 
| Previous Message | Eugen Konkov | 2019-11-07 09:20:32 | Re: Does 'instead of delete' trigger support modification of OLD | 
| From | Date | Subject | |
|---|---|---|---|
| Next Message | 曾文旌 (义从) | 2019-11-07 09:30:27 | Re: [Proposal] Global temporary tables | 
| Previous Message | Grigory Smolkin | 2019-11-07 09:22:28 | Re: [proposal] recovery_target "latest" |