Content-Type: multipart/alternative;
 boundary="------------7ABa4WQGdwYvQ7H2YTqb0t2C"
Message-ID: <2a3a38fb-818f-491f-bcd5-0c0672c874f2@dunslane.net>
Date: Wed, 19 Feb 2025 17:18:58 -0500
MIME-Version: 1.0
User-Agent: Mozilla Thunderbird
Subject: Re: jsonb_strip_nulls with arrays?
To: Florents Tselai <florents.tselai@gmail.com>
Cc: pgsql-hackers <pgsql-hackers@lists.postgresql.org>
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>
 <FBED3D5A-5AF7-40B5-BEDF-51842B0194C4@gmail.com>
 <F4E9D398-399E-46C1-ABC1-725B7E49261C@gmail.com>
From: Andrew Dunstan <andrew@dunslane.net>
Content-Language: en-US
Autocrypt: addr=andrew@dunslane.net; keydata=
 xsBNBE7KWFkBCAClridxur2AIc7eW2AR7izbfp3EnNefie2HbLF0izW5Ik5UjX2HBXBx4syI
 gY6b0ugohXrr274+baoAlvSbq6cAoQuEVrk5IZFzt20b1Xkx65FwGSEj526yiKLocqkJceSq
 Xr9xcA5SGY+FZv441chh5SU92v4q6z+6LPpoHOh97ptAVXZYNTtU0LevyvD5lja0TzbvJm6C
 eFXitJfnm1pLEr0DGJCR/iUOl/N62Kh4855zZC7NHIjQHPOvV5Stz/l5ilDhvGVk+xkXFPys
 SjZoUr1rXhYLpiyi5sR0X9FHXT0KnGuz1F5ERO7ZTLSSQ6fJwPj6gOk9K+vvoKvoeql5ABEB
 AAHNJEFuZHJldyBEdW5zdGFuIDxhbmRyZXdAZHVuc2xhbmUubmV0PsLAmwQTAQgARQIbAwIX
 gAIZAQULCQgHAgMiAgEGFQoJCAsCBBYCAwECHgcWIQTkPlhGHfx8v0RpFaWZ+n/LWfw7gQUC
 ZFlxxwUJGVGAbgAKCRCZ+n/LWfw7gXikB/9ZdcUy6CTBFIIuL/bVsc1eLEW/gJBjJBF6HxNY
 xgEkAgXAp4Lg4A5U+QB9GouFr7+GYxF0BU4hzoGhNPUWltxnHdMWP8nC/38LAqgMi8L/bbsm
 HW5YPBdWYaAZAPJQVfOAgjTbRUb26KSprpyrrJKW0ZmrZfjhNPcQ72jpWzoPLQqx2X6B0fru
 1jq+cBh8lb6r1mJTim1T3JIn+F/v5VpdQS+EL8xqsHkfzKjIPsW3CIXpkypSk6saA55Rkkbl
 26AW8ftPVB0Q6Lnn6FLt9CP0MGNixBQ55yq8r1K+nCBvCCjvQjM8RDm0UUum0WNl+ifQgTLO
 E8TWEnwVtkBf+3QWzsBNBE7KWFkBCADRnOM0FCzsYW6jtncg+dWIagjUZpvaClmqn/sJluLa
 Q3v1VXMQJzYs3eC1gh386W+XBwLRpDj3jzH81lX+p73Re3d3oJW7X+ffsxuzu5ZVdMUkqBYo
 nkAbKxr6gyJ12F/+JkUVzLcoTN+d/7YsQvUVi7NaKH8mJgjz112O4fUe3p9wfAaFa0RXHc5S
 GPzRTYRRlv/XZBIho4J2tkZOnteZJZ+GbxQVlINt6fd8P6al3MWOvpP/ExJPguEfjOsO6Njy
 xjo3WfpD4lHMOR/Oc3/8mScEF84rF2jXbsFgelWnbPWAvXY+pD0dXOFRkagGmC/viwBDqq5b
 5tk76kKmUbZxABEBAAHCwHwEGAEIACYCGwwWIQTkPlhGHfx8v0RpFaWZ+n/LWfw7gQUCZFlx
 5wUJGVGAjgAKCRCZ+n/LWfw7gf+iB/4g8CPY5jihf5r/8EsoIGe2H+dpVmpPF8YGBzTIvCz/
 fQoOq8AX/pE76QEuFnFZWfjw+wgBXgCVmkox2Eflkk6z4ND3pcwGZ6CfCxTQCDk/dij+2DQ4
 6bmDCy/sBgcbz9mTpoLC11HLoPae6YN9nBNQRZDcEFEu54OaVOqlIdbA6m+POIBCXZdHOFc0
 WoDTgxHRzC1jgQNidyd6tKqcsVJs0dzF0oKTmFFmUAqTdJO12LBuNA1rlqrR3EtpYk8B/wtS
 5dIMD7Q8hwQpL+4C6GNpb6ZKnPkLi47pDOLhz2qBrqN+rqUEsT3YnExYpzj5yOBi+FlmV1Hw
 49QYe1sn2ZPs
In-Reply-To: <F4E9D398-399E-46C1-ABC1-725B7E49261C@gmail.com>
Archived-At: <https://www.postgresql.org/message-id/2a3a38fb-818f-491f-bcd5-0c0672c874f2%40dunslane.net>
Precedence: bulk

This is a multi-part message in MIME format.
--------------7ABa4WQGdwYvQ7H2YTqb0t2C
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 8bit


On 2025-02-19 We 4:23 PM, Florents Tselai wrote:
>
>
>> On 18 Jan 2025, at 11:51 AM, Florents Tselai 
>> <florents.tselai@gmail.com> wrote:
>>
>>
>>
>>> On 8 Jan 2025, at 6:45 PM, Andrew Dunstan <andrew@dunslane.net> wrote:
>>>
>>>
>>> On 2024-09-17 Tu 4:53 PM, Florents Tselai wrote:
>>>>
>>>>     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.
>>>>
>>>> Here's a patch that adds that argument (only for jsonb; no json 
>>>> implementation yet)
>>>>
>>>>
>>>
>>> I think it looks sane. We're not stripping a top level null, which 
>>> is one thing I looked out for.
>>>
>>> I think we need a json implementation as well, though.
>>>
>>>
>> Thanks for having a Look, Andrew;
>> if there aren’t any other objections, I’ll come back with a json 
>> implementation too.
>
> Attached is a v2 patch with the missing json implementation.
> jsonb one remains the same.


Please add this to the next Commitfest at 
https://commitfest.postgresql.org/52/


cheers


andrew


>
>
>
--
Andrew Dunstan
EDB:https://www.enterprisedb.com

--------------7ABa4WQGdwYvQ7H2YTqb0t2C
Content-Type: text/html; charset=UTF-8
Content-Transfer-Encoding: 8bit

<!DOCTYPE html>
<html>
  <head>
    <meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
  </head>
  <body>
    <p><br>
    </p>
    <div class="moz-cite-prefix">On 2025-02-19 We 4:23 PM, Florents
      Tselai wrote:<br>
    </div>
    <blockquote type="cite"
      cite="mid:F4E9D398-399E-46C1-ABC1-725B7E49261C@gmail.com">
      <meta http-equiv="content-type" content="text/html; charset=UTF-8">
      <br id="lineBreakAtBeginningOfMessage">
      <div><br>
        <blockquote type="cite">
          <div>On 18 Jan 2025, at 11:51 AM, Florents Tselai
            <a class="moz-txt-link-rfc2396E" href="mailto:florents.tselai@gmail.com">&lt;florents.tselai@gmail.com&gt;</a> wrote:</div>
          <br class="Apple-interchange-newline">
          <div>
            <meta charset="UTF-8">
            <div
style="caret-color: rgb(0, 0, 0); font-family: Helvetica; font-size: 14px; font-style: normal; font-variant-caps: normal; font-weight: 400; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-decoration: none;"><br
                class="Apple-interchange-newline">
              <br>
              <blockquote type="cite">
                <div>On 8 Jan 2025, at 6:45 PM, Andrew Dunstan
                  <a class="moz-txt-link-rfc2396E" href="mailto:andrew@dunslane.net">&lt;andrew@dunslane.net&gt;</a> wrote:</div>
                <br class="Apple-interchange-newline">
                <div>
                  <div>
                    <p><br>
                    </p>
                    <div class="moz-cite-prefix">On 2024-09-17 Tu 4:53
                      PM, Florents Tselai wrote:<br>
                    </div>
                    <blockquote type="cite"
cite="mid:CA+v5N43=48Ddg=Ub313bX3g2qu9VRHzfTP3TAGN10WF6XNX_2w@mail.gmail.com">
                      <div dir="ltr"><br>
                        <div class="gmail_quote">
                          <blockquote class="gmail_quote"
style="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; </div>
            <div
style="caret-color: rgb(0, 0, 0); font-family: Helvetica; font-size: 14px; font-style: normal; font-variant-caps: normal; font-weight: 400; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-decoration: none;">if
              there aren’t any other objections, I’ll come back with a
              json implementation too. <br>
            </div>
          </div>
        </blockquote>
        <div><br>
        </div>
        <div>Attached is a v2 patch with the missing json
          implementation. </div>
        <div>jsonb one remains the same.</div>
      </div>
    </blockquote>
    <p><br>
    </p>
    <p><br>
    </p>
    <p>Please add this to the next Commitfest at
      <a class="moz-txt-link-freetext" href="https://commitfest.postgresql.org/52/">https://commitfest.postgresql.org/52/</a></p>
    <p><br>
    </p>
    <p>cheers</p>
    <p><br>
    </p>
    <p>andrew<br>
    </p>
    <p><br>
    </p>
    <blockquote type="cite"
      cite="mid:F4E9D398-399E-46C1-ABC1-725B7E49261C@gmail.com">
      <div>
        <div><br>
        </div>
      </div>
      <br>
      <fieldset class="moz-mime-attachment-header"></fieldset>
      <meta http-equiv="content-type" content="text/html; charset=UTF-8">
      <meta http-equiv="content-type" content="text/html; charset=UTF-8">
      <br>
      <fieldset class="moz-mime-attachment-header"></fieldset>
    </blockquote>
    <pre class="moz-signature"
    signature-switch-id="d0437855-2267-4610-80b3-83167ec45b0b" cols="72">--
Andrew Dunstan
EDB: <a class="moz-txt-link-freetext" href="https://www.enterprisedb.com">https://www.enterprisedb.com</a>
</pre>
  </body>
</html>

--------------7ABa4WQGdwYvQ7H2YTqb0t2C--