MIME-Version: 1.0
References: <19516-97e045ee5e78c98d@postgresql.org>
In-Reply-To: <19516-97e045ee5e78c98d@postgresql.org>
From: Marko Grujic <markoog@gmail.com>
Date: Wed, 10 Jun 2026 12:50:39 +0200
Message-ID: 
 <CAKQrOnpOSFrQ=cdSeHfoBFOwNT3=aa55t2VRZMt_xeOf9dMO8Q@mail.gmail.com>
Subject: Re: BUG #19516: Mixup of OLD/NEW with RETURNING when parenthesized
To: pgsql-bugs@lists.postgresql.org
Content-Type: multipart/alternative; boundary="0000000000005911e00653e4045e"
Archived-At: 
 <https://www.postgresql.org/message-id/CAKQrOnpOSFrQ%3DcdSeHfoBFOwNT3%3Daa55t2VRZMt_xeOf9dMO8Q%40mail.gmail.com>
Precedence: bulk

--0000000000005911e00653e4045e
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

Proposed patch:
https://www.postgresql.org/message-id/CAOvwyF2cO_5mAt%3Dw%3Dy-dFnaG5UkZ%2B3=
H8nSDoKF_iuWZHsU2ARg%40mail.gmail.com

On Wed, Jun 10, 2026 at 12:10=E2=80=AFPM PG Bug reporting form <
noreply@postgresql.org> wrote:

> The following bug has been logged on the website:
>
> Bug reference:      19516
> Logged by:          Marko Grujic
> Email address:      markoog@gmail.com
> PostgreSQL version: 18.4
> Operating system:   MacOS, Debian
> Description:
>
> Hi, reporting a basic bug which involves a RETURNING clause on DML with
> parenthesized OLD/NEW.
>
> Effectively, the (parenthesized) row reference isn't being honored, and
> instead the default behavior is applied.
>
> Seeing this on tip of master as well (19beta1).
>
> postgres=3D# create table t(a int, b text);
> CREATE TABLE
> postgres=3D# insert into t values (1, 'one') returning (old).b, old.b,
> (new).b, new.b;
>   b  | b |  b  |  b
> -----+---+-----+-----
>  one |   | one | one
> (1 row)
>
> INSERT 0 1
> postgres=3D# update t set b =3D '1' where a =3D 1 returning (old).b, old.=
b,
> (new).b, new.b;
>  b |  b  | b | b
> ---+-----+---+---
>  1 | one | 1 | 1
> (1 row)
>
> UPDATE 1
> postgres=3D# delete from t where a =3D 1 returning (old).b, old.b, (new).=
b,
> new.b;
>  b | b | b | b
> ---+---+---+---
>  1 | 1 | 1 |
> (1 row)
>
> DELETE 1
> postgres=3D# select version();
>                                                       version
>
> -------------------------------------------------------------------------=
-------------------------------------------
>  PostgreSQL 18.4 (Debian 18.4-1.pgdg13+1) on x86_64-pc-linux-gnu, compile=
d
> by gcc (Debian 14.2.0-19) 14.2.0, 64-bit
> (1 row)
>
>
>

--0000000000005911e00653e4045e
Content-Type: text/html; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

<div dir=3D"ltr">Proposed patch: <a href=3D"https://www.postgresql.org/mess=
age-id/CAOvwyF2cO_5mAt%3Dw%3Dy-dFnaG5UkZ%2B3H8nSDoKF_iuWZHsU2ARg%40mail.gma=
il.com">https://www.postgresql.org/message-id/CAOvwyF2cO_5mAt%3Dw%3Dy-dFnaG=
5UkZ%2B3H8nSDoKF_iuWZHsU2ARg%40mail.gmail.com</a></div><br><div class=3D"gm=
ail_quote gmail_quote_container"><div dir=3D"ltr" class=3D"gmail_attr">On W=
ed, Jun 10, 2026 at 12:10=E2=80=AFPM PG Bug reporting form &lt;<a href=3D"m=
ailto:noreply@postgresql.org">noreply@postgresql.org</a>&gt; wrote:<br></di=
v><blockquote class=3D"gmail_quote" style=3D"margin:0px 0px 0px 0.8ex;borde=
r-left:1px solid rgb(204,204,204);padding-left:1ex">The following bug has b=
een logged on the website:<br>
<br>
Bug reference:=C2=A0 =C2=A0 =C2=A0 19516<br>
Logged by:=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 Marko Grujic<br>
Email address:=C2=A0 =C2=A0 =C2=A0 <a href=3D"mailto:markoog@gmail.com" tar=
get=3D"_blank">markoog@gmail.com</a><br>
PostgreSQL version: 18.4<br>
Operating system:=C2=A0 =C2=A0MacOS, Debian<br>
Description:=C2=A0 =C2=A0 =C2=A0 =C2=A0 <br>
<br>
Hi, reporting a basic bug which involves a RETURNING clause on DML with<br>
parenthesized OLD/NEW.<br>
<br>
Effectively, the (parenthesized) row reference isn&#39;t being honored, and=
<br>
instead the default behavior is applied.<br>
<br>
Seeing this on tip of master as well (19beta1).<br>
<br>
postgres=3D# create table t(a int, b text);<br>
CREATE TABLE<br>
postgres=3D# insert into t values (1, &#39;one&#39;) returning (old).b, old=
.b,<br>
(new).b, new.b;<br>
=C2=A0 b=C2=A0 | b |=C2=A0 b=C2=A0 |=C2=A0 b<br>
-----+---+-----+-----<br>
=C2=A0one |=C2=A0 =C2=A0| one | one<br>
(1 row)<br>
<br>
INSERT 0 1<br>
postgres=3D# update t set b =3D &#39;1&#39; where a =3D 1 returning (old).b=
, old.b,<br>
(new).b, new.b;<br>
=C2=A0b |=C2=A0 b=C2=A0 | b | b<br>
---+-----+---+---<br>
=C2=A01 | one | 1 | 1<br>
(1 row)<br>
<br>
UPDATE 1<br>
postgres=3D# delete from t where a =3D 1 returning (old).b, old.b, (new).b,=
<br>
new.b;<br>
=C2=A0b | b | b | b<br>
---+---+---+---<br>
=C2=A01 | 1 | 1 |<br>
(1 row)<br>
<br>
DELETE 1<br>
postgres=3D# select version();<br>
=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=
=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =
=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 version<br>
---------------------------------------------------------------------------=
-----------------------------------------<br>
=C2=A0PostgreSQL 18.4 (Debian 18.4-1.pgdg13+1) on x86_64-pc-linux-gnu, comp=
iled<br>
by gcc (Debian 14.2.0-19) 14.2.0, 64-bit<br>
(1 row)<br>
<br>
<br>
</blockquote></div>

--0000000000005911e00653e4045e--