MIME-Version: 1.0
References: <19409-e16cd2605e59a4af@postgresql.org> <562481.1771013899@sss.pgh.pa.us>
In-Reply-To: <562481.1771013899@sss.pgh.pa.us>
From: Florents Tselai <florents.tselai@gmail.com>
Date: Fri, 13 Feb 2026 23:35:04 +0200
Message-ID: <CA+v5N43CCjEtORJdn3Q0+uh=ztorEiiUnWBVc2TNci4zJ57ntQ@mail.gmail.com>
Subject: Re: BUG #19409: Function jsonb_strip_nulls() changed from immutable
 to stable.
To: Tom Lane <tgl@sss.pgh.pa.us>
Cc: lucio.chiessi@trustly.com, pgsql-bugs@lists.postgresql.org, 
	Andrew Dunstan <andrew@dunslane.net>
Content-Type: multipart/mixed; boundary="000000000000168661064abb630e"
Archived-At: <https://www.postgresql.org/message-id/CA%2Bv5N43CCjEtORJdn3Q0%2Buh%3DztorEiiUnWBVc2TNci4zJ57ntQ%40mail.gmail.com>
Precedence: bulk

--000000000000168661064abb630e
Content-Type: multipart/alternative; boundary="00000000000016865f064abb630c"

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

On Fri, Feb 13, 2026 at 10:18=E2=80=AFPM Tom Lane <tgl@sss.pgh.pa.us> wrote=
:

> PG Bug reporting form <noreply@postgresql.org> writes:
> > ... this fails in PostgreSQL 18, because the jsonb_strip_nulls ( target
> > jsonb [,strip_in_arrays boolean ] ) function changed from immutable to
> > stable.
>
> A bit of git excavation shows that this changed here:
>
> Author: Andrew Dunstan <andrew@dunslane.net>
> Branch: master Release: REL_18_BR [4603903d2] 2025-03-05 10:04:02 -0500
>
>     Allow json{b}_strip_nulls to remove null array elements
>
>     An additional paramater ("strip_in_arrays") is added to these
> functions.
>     It defaults to false. If true, then null array elements are removed a=
s
>     well as null valued object fields. JSON that just consists of a singl=
e
>     null is not affected.
>
>     Author: Florents Tselai <florents.tselai@gmail.com>
>
>     Discussion:
> https://postgr.es/m/4BCECCD5-4F40-4313-9E98-9E16BEB0B01D@gmail.com
>
> It looks like a thinko to me, because surely the strip_in_arrays
> parameter did not make the function more mutable than before.
>

The perils of going from pg_proc.dat to manual defs in system_functions.sql

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

<div dir=3D"ltr"><div dir=3D"ltr"><div dir=3D"ltr"><div><br></div></div><br=
><div class=3D"gmail_quote"><div dir=3D"ltr" class=3D"gmail_attr">On Fri, F=
eb 13, 2026 at 10:18=E2=80=AFPM Tom Lane &lt;<a href=3D"mailto:tgl@sss.pgh.=
pa.us" target=3D"_blank">tgl@sss.pgh.pa.us</a>&gt; wrote:<br></div><blockqu=
ote class=3D"gmail_quote" style=3D"margin:0px 0px 0px 0.8ex;border-left-wid=
th:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-l=
eft:1ex">PG Bug reporting form &lt;<a href=3D"mailto:noreply@postgresql.org=
" target=3D"_blank">noreply@postgresql.org</a>&gt; writes:<br>
&gt; ... this fails in PostgreSQL 18, because the jsonb_strip_nulls ( targe=
t<br>
&gt; jsonb [,strip_in_arrays boolean ] ) function changed from immutable to=
<br>
&gt; stable.<br>
<br>
A bit of git excavation shows that this changed here:<br>
<br>
Author: Andrew Dunstan &lt;<a href=3D"mailto:andrew@dunslane.net" target=3D=
"_blank">andrew@dunslane.net</a>&gt;<br>
Branch: master Release: REL_18_BR [4603903d2] 2025-03-05 10:04:02 -0500<br>
<br>
=C2=A0 =C2=A0 Allow json{b}_strip_nulls to remove null array elements<br>
<br>
=C2=A0 =C2=A0 An additional paramater (&quot;strip_in_arrays&quot;) is adde=
d to these functions.<br>
=C2=A0 =C2=A0 It defaults to false. If true, then null array elements are r=
emoved as<br>
=C2=A0 =C2=A0 well as null valued object fields. JSON that just consists of=
 a single<br>
=C2=A0 =C2=A0 null is not affected.<br>
<br>
=C2=A0 =C2=A0 Author: Florents Tselai &lt;<a href=3D"mailto:florents.tselai=
@gmail.com" target=3D"_blank">florents.tselai@gmail.com</a>&gt;<br>
<br>
=C2=A0 =C2=A0 Discussion: <a href=3D"https://postgr.es/m/4BCECCD5-4F40-4313=
-9E98-9E16BEB0B01D@gmail.com" rel=3D"noreferrer" target=3D"_blank">https://=
postgr.es/m/4BCECCD5-4F40-4313-9E98-9E16BEB0B01D@gmail.com</a><br>
<br>
It looks like a thinko to me, because surely the strip_in_arrays<br>
parameter did not make the function more mutable than before.<br></blockquo=
te><div><br></div><div>The perils of going from pg_proc.dat to manual defs =
in system_functions.sql =C2=A0</div></div></div>
</div>

--00000000000016865f064abb630c--
--000000000000168661064abb630e
Content-Type: application/octet-stream; 
	name="v1-0001-Restore-json-b-_strip_nulls-immutability.patch"
Content-Disposition: attachment; 
	filename="v1-0001-Restore-json-b-_strip_nulls-immutability.patch"
Content-Transfer-Encoding: base64
Content-ID: <f_mllen6zo0>
X-Attachment-Id: f_mllen6zo0

RnJvbSAwODhlNmYyNDAwYzUwMDA3ZTYzYjVmOGIxOGZiNjAwNzVjYzc4YzQ2IE1vbiBTZXAgMTcg
MDA6MDA6MDAgMjAwMQpGcm9tOiBGbG9yZW50cyBUc2VsYWkgPGZsb3JlbnRzLnRzZWxhaUBnbWFp
bC5jb20+CkRhdGU6IEZyaSwgMTMgRmViIDIwMjYgMjI6NTM6MTQgKzAyMDAKU3ViamVjdDogW1BB
VENIIHYxXSBSZXN0b3JlIGpzb257Yn1fc3RyaXBfbnVsbHMgaW1tdXRhYmlsaXR5CgpDb21taXQ0
NjAzOTAzZDIgYWRkZWQgdGhlIHN0cmlwX2luX2FycmF5cyBwYXJhbWV0ZXIgdG8gdGhlc2UgZnVu
Y3Rpb25zCmJ1dCBtaXN0YWtlbmx5IGNoYW5nZWQgdGhlaXIgdm9sYXRpbGl0eSBmcm9tIElNTVVU
QUJMRSB0byBTVEFCTEUuIFRoaXMKY29tbWl0IHJlc3RvcmVzIHRoZSBjb3JyZWN0IElNTVVUQUJM
RSBtYXJraW5nLCBhcyB0aGUgZnVuY3Rpb25zJyBvdXRwdXQKc3RpbGwgZGVwZW5kcyBlbnRpcmVs
eSBvbiB0aGVpciBpbnB1dCBhcmd1bWVudHMuCi0tLQogc3JjL2JhY2tlbmQvY2F0YWxvZy9zeXN0
ZW1fZnVuY3Rpb25zLnNxbCB8IDQgKystLQogMSBmaWxlIGNoYW5nZWQsIDIgaW5zZXJ0aW9ucygr
KSwgMiBkZWxldGlvbnMoLSkKCmRpZmYgLS1naXQgYS9zcmMvYmFja2VuZC9jYXRhbG9nL3N5c3Rl
bV9mdW5jdGlvbnMuc3FsIGIvc3JjL2JhY2tlbmQvY2F0YWxvZy9zeXN0ZW1fZnVuY3Rpb25zLnNx
bAppbmRleCBlYjllMzFhZTFiZi4uODM2YTBmOWViM2QgMTAwNjQ0Ci0tLSBhL3NyYy9iYWNrZW5k
L2NhdGFsb2cvc3lzdGVtX2Z1bmN0aW9ucy5zcWwKKysrIGIvc3JjL2JhY2tlbmQvY2F0YWxvZy9z
eXN0ZW1fZnVuY3Rpb25zLnNxbApAQCAtNjExLDE0ICs2MTEsMTQgQEAgQ1JFQVRFIE9SIFJFUExB
Q0UgRlVOQ1RJT04KICAganNvbmJfc3RyaXBfbnVsbHModGFyZ2V0IGpzb25iLCBzdHJpcF9pbl9h
cnJheXMgYm9vbGVhbiBERUZBVUxUIGZhbHNlKQogUkVUVVJOUyBqc29uYgogTEFOR1VBR0UgSU5U
RVJOQUwKLVNUUklDVCBTVEFCTEUgUEFSQUxMRUwgU0FGRQorU1RSSUNUIElNTVVUQUJMRSBQQVJB
TExFTCBTQUZFCiBBUyAnanNvbmJfc3RyaXBfbnVsbHMnOwogCiBDUkVBVEUgT1IgUkVQTEFDRSBG
VU5DVElPTgogICBqc29uX3N0cmlwX251bGxzKHRhcmdldCBqc29uLCBzdHJpcF9pbl9hcnJheXMg
Ym9vbGVhbiBERUZBVUxUIGZhbHNlKQogUkVUVVJOUyBqc29uCiBMQU5HVUFHRSBJTlRFUk5BTAot
U1RSSUNUIFNUQUJMRSBQQVJBTExFTCBTQUZFCitTVFJJQ1QgSU1NVVRBQkxFIFBBUkFMTEVMIFNB
RkUKIEFTICdqc29uX3N0cmlwX251bGxzJzsKIAogLS0gZGVmYXVsdCBub3JtYWxpemF0aW9uIGZv
cm0gaXMgTkZDLCBwZXIgU1FMIHN0YW5kYXJkCi0tIAoyLjUyLjAKCg==
--000000000000168661064abb630e--