Re: INS/UPD/DEL RETURNING for 8.2 - Mailing list pgsql-patches

From Jonah H. Harris
Subject Re: INS/UPD/DEL RETURNING for 8.2
Date
Msg-id 36e682920603021748x5a597ch5f2ee0d756fc2bc@mail.gmail.com
Whole thread Raw
In response to Re: INS/UPD/DEL RETURNING for 8.2  (Tom Lane <tgl@sss.pgh.pa.us>)
Responses Re: INS/UPD/DEL RETURNING for 8.2  ("Jonah H. Harris" <jonah.harris@gmail.com>)
List pgsql-patches
On 3/2/06, Tom Lane <tgl@sss.pgh.pa.us> wrote:
"Jonah H. Harris" <jonah.harris@gmail.com> writes:
> INSERT/UPDATE/DELETE seem to work fine in normal operation but there is an
> error with DELETE RETURNING when used through PL/pgSQL.

Probably other places too.  I don't see any provision here for ensuring
that the variables used in the RETURNING list are actually computed by
the plan.  This would be masked in the INSERT and non-join UPDATE cases
by the fact that the plan has to compute all columns of the target table
anyway ... but in a DELETE it'd be an issue.

I think set-returning functions in the RETURNING list might give you
some fits too ...

Yeah, I got to looking into the special tuple handling code in execUtils for retrieving the old (deleted) tuple and there's something definitely getting lost along the way in some cases.

--
Jonah H. Harris, Database Internals Architect
EnterpriseDB Corporation
732.331.1324

pgsql-patches by date:

Previous
From: "Jonah H. Harris"
Date:
Subject: Re: INS/UPD/DEL RETURNING for 8.2
Next
From: Bruce Momjian
Date:
Subject: Re: [HACKERS] to_char and i18n