From: Florents Tselai <florents.tselai@gmail.com>
Message-Id: <FBED3D5A-5AF7-40B5-BEDF-51842B0194C4@gmail.com>
Content-Type: multipart/alternative;
	boundary="Apple-Mail=_CAFC83C3-71FB-4CA7-B6C0-67BE29A50BD0"
Mime-Version: 1.0 (Mac OS X Mail 16.0 \(3826.300.87.4.3\))
Subject: Re: jsonb_strip_nulls with arrays?
Date: Sat, 18 Jan 2025 11:51:22 +0200
In-Reply-To: <00e32809-ae7e-46cf-9b33-7323f5fc7784@dunslane.net>
Cc: pgsql-hackers <pgsql-hackers@lists.postgresql.org>
To: Andrew Dunstan <andrew@dunslane.net>
References: <4BCECCD5-4F40-4313-9E98-9E16BEB0B01D@gmail.com>
 <92de2543-de33-4092-9de7-b532a078353f@dunslane.net>
 <CA+v5N43=48Ddg=Ub313bX3g2qu9VRHzfTP3TAGN10WF6XNX_2w@mail.gmail.com>
 <00e32809-ae7e-46cf-9b33-7323f5fc7784@dunslane.net>
Archived-At: <https://www.postgresql.org/message-id/FBED3D5A-5AF7-40B5-BEDF-51842B0194C4%40gmail.com>
Precedence: bulk


--Apple-Mail=_CAFC83C3-71FB-4CA7-B6C0-67BE29A50BD0
Content-Transfer-Encoding: quoted-printable
Content-Type: text/plain;
	charset=utf-8


> On 8 Jan 2025, at 6:45=E2=80=AFPM, Andrew Dunstan =
<andrew@dunslane.net> wrote:
>=20
>=20
>=20
> On 2024-09-17 Tu 4:53 PM, Florents Tselai wrote:
>>=20
>>> We could, if we're going to do anything at all in this area. Another =
possibility would be to provide a second optional parameter for =
json{b}_strip_nulls. That's probably a better way to go.
>>>=20
>> Here's a patch that adds that argument (only for jsonb; no json =
implementation yet)
>>=20
>>=20
>=20
> I think it looks sane. We're not stripping a top level null, which is =
one thing I looked out for.
>=20
> I think we need a json implementation as well, though.
>=20
>=20
>=20
Thanks for having a Look, Andrew;=20
if there aren=E2=80=99t any other objections, I=E2=80=99ll come back =
with a json implementation too.=20
> cheers
>=20
>=20
>=20
> andrew
>=20
>=20
>=20
> --
> Andrew Dunstan
> EDB: https://www.enterprisedb.com <https://www.enterprisedb.com/>


--Apple-Mail=_CAFC83C3-71FB-4CA7-B6C0-67BE29A50BD0
Content-Transfer-Encoding: quoted-printable
Content-Type: text/html;
	charset=utf-8

<html><head><meta http-equiv=3D"content-type" content=3D"text/html; =
charset=3Dutf-8"></head><body style=3D"overflow-wrap: break-word; =
-webkit-nbsp-mode: space; line-break: after-white-space;"><br =
id=3D"lineBreakAtBeginningOfMessage"><div><br><blockquote =
type=3D"cite"><div>On 8 Jan 2025, at 6:45=E2=80=AFPM, Andrew Dunstan =
&lt;andrew@dunslane.net&gt; wrote:</div><br =
class=3D"Apple-interchange-newline"><div>

 =20
    <meta http-equiv=3D"Content-Type" content=3D"text/html; =
charset=3DUTF-8">
 =20
  <div><p><br>
    </p>
    <div class=3D"moz-cite-prefix">On 2024-09-17 Tu 4:53 PM, Florents
      Tselai wrote:<br>
    </div>
    <blockquote type=3D"cite" =
cite=3D"mid:CA+v5N43=3D48Ddg=3DUb313bX3g2qu9VRHzfTP3TAGN10WF6XNX_2w@mail.g=
mail.com">
      <meta http-equiv=3D"content-type" content=3D"text/html; =
charset=3DUTF-8">
      <div dir=3D"ltr"><br>
        <div class=3D"gmail_quote">
          <blockquote class=3D"gmail_quote" style=3D"margin:0px 0px 0px =
0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(=
204,204,204);padding-left:1ex">
            <div><p>We could, if we're going to do anything at all in =
this
                area. Another possibility would be to provide a second
                optional parameter for json{b}_strip_nulls. That's
                probably a better way to go.</p>
            </div>
          </blockquote>
          <div>Here's a patch that adds that argument (only for jsonb;
            no json implementation yet)</div>
          <div><br>
          </div>
          <br>
        </div>
      </div>
    </blockquote><p><br>
    </p><p>I think it looks sane. We're not stripping a top level null,
      which is one thing I looked out for.</p><p>I think we need a json =
implementation as well, =
though.</p><p><br></p></div></div></blockquote>Thanks for having a Look, =
Andrew;&nbsp;</div><div>if there aren=E2=80=99t any other objections, =
I=E2=80=99ll come back with a json implementation =
too.&nbsp;<br><blockquote type=3D"cite"><div><div><p>
    </p><p>cheers</p><p><br>
    </p><p>andrew<br>
    </p><p><br>
    </p>
    <pre class=3D"moz-signature" =
signature-switch-id=3D"d0437855-2267-4610-80b3-83167ec45b0b" =
cols=3D"72">--
Andrew Dunstan
EDB: <a class=3D"moz-txt-link-freetext" =
href=3D"https://www.enterprisedb.com/">https://www.enterprisedb.com</a>
</pre>
  </div>

</div></blockquote></div><br></body></html>=

--Apple-Mail=_CAFC83C3-71FB-4CA7-B6C0-67BE29A50BD0--