MIME-Version: 1.0
References: <CACxu=vJaKFNsYxooSnW1wEgsAO5u_v1XYBacfVJ14wgJV_PYeg@mail.gmail.com>
 <1342498.1729444411@sss.pgh.pa.us> <CACxu=vLXvpzN4X3k+9jsMt6ujuOvFVUSkA80t_cROSsF4y2jQQ@mail.gmail.com>
 <1445998.1729482404@sss.pgh.pa.us> <CACxu=vKEF8Qa-OaADFxf0uMg-xw6gH_CNCWd2s+xaqh-gY4=xg@mail.gmail.com>
 <2062830.1729625620@sss.pgh.pa.us> <CACxu=v+t5xDh0K=tOVmFrumvAH60izWhB16a9k7f2A8jPxQEaw@mail.gmail.com>
 <2259890.1729696885@sss.pgh.pa.us> <CACxu=vJMqLUZR1N-AT4cXw9JHMsN=5-iYvPvKWy-CcKwz_9aBg@mail.gmail.com>
 <2351536.1729735831@sss.pgh.pa.us> <CACxu=vK9zZRVasn3Pmn3tbe8z715rPWiXtQ=rKzF9_KbUcH2QA@mail.gmail.com>
 <1947217.1730499609@sss.pgh.pa.us>
In-Reply-To: <1947217.1730499609@sss.pgh.pa.us>
From: Michel Pelletier <pelletier.michel@gmail.com>
Date: Fri, 1 Nov 2024 17:50:52 -0700
Message-ID: <CACxu=vKxR_1RGrjppLDjnhRj4Sd-QEMeqt+2gPz0myK8kq6q0Q@mail.gmail.com>
Subject: Re: Using Expanded Objects other than Arrays from plpgsql
To: Tom Lane <tgl@sss.pgh.pa.us>
Cc: pgsql-hackers@lists.postgresql.org
Content-Type: multipart/alternative; boundary="000000000000b6a4040625e3734f"
Archived-At: <https://www.postgresql.org/message-id/CACxu%3DvKxR_1RGrjppLDjnhRj4Sd-QEMeqt%2B2gPz0myK8kq6q0Q%40mail.gmail.com>
Precedence: bulk

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

On Fri, Nov 1, 2024 at 3:20=E2=80=AFPM Tom Lane <tgl@sss.pgh.pa.us> wrote:

> Michel Pelletier <pelletier.michel@gmail.com> writes:
>
> Well, you shouldn't be using PERFORM.  Not only does it not do the
> right thing, but it's not optimized for expanded objects at all:
> they'll get flattened both on the way into the statement and on
> the way out.  Try it with
>
>      graph :=3D set_element(graph, 1, 1, 1);
>      RETURN nvals(graph);
>

Ah my bad, you mentioned that already and I missed it, here's the two
backtraces with the assignment:

https://gist.githubusercontent.com/michelp/20b917686149d482be2359569845f232=
/raw/ca8349ae4b0469674b4b2c77f34c5a02553583a9/gistfile1.txt


>
>                         regards, tom lane
>

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

<div dir=3D"ltr"><div dir=3D"ltr">On Fri, Nov 1, 2024 at 3:20=E2=80=AFPM To=
m Lane &lt;<a href=3D"mailto:tgl@sss.pgh.pa.us">tgl@sss.pgh.pa.us</a>&gt; w=
rote:<br></div><div class=3D"gmail_quote"><blockquote class=3D"gmail_quote"=
 style=3D"margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);p=
adding-left:1ex">Michel Pelletier &lt;<a href=3D"mailto:pelletier.michel@gm=
ail.com" target=3D"_blank">pelletier.michel@gmail.com</a>&gt; writes:<br><b=
r>
Well, you shouldn&#39;t be using PERFORM.=C2=A0 Not only does it not do the=
<br>
right thing, but it&#39;s not optimized for expanded objects at all:<br>
they&#39;ll get flattened both on the way into the statement and on<br>
the way out.=C2=A0 Try it with<br>
<br>
=C2=A0 =C2=A0 =C2=A0graph :=3D set_element(graph, 1, 1, 1);<br>
=C2=A0 =C2=A0 =C2=A0RETURN nvals(graph);<br></blockquote><div><br></div><di=
v>Ah my bad, you mentioned that already and I missed it, here&#39;s the two=
 backtraces with the assignment:</div><div><br></div><div><a href=3D"https:=
//gist.githubusercontent.com/michelp/20b917686149d482be2359569845f232/raw/c=
a8349ae4b0469674b4b2c77f34c5a02553583a9/gistfile1.txt">https://gist.githubu=
sercontent.com/michelp/20b917686149d482be2359569845f232/raw/ca8349ae4b04696=
74b4b2c77f34c5a02553583a9/gistfile1.txt</a></div><div>=C2=A0</div><blockquo=
te class=3D"gmail_quote" style=3D"margin:0px 0px 0px 0.8ex;border-left:1px =
solid rgb(204,204,204);padding-left:1ex">
<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 regards, tom lane<br>
</blockquote></div></div>

--000000000000b6a4040625e3734f--