From: | Yan Chengpeng <chengpeng_yan(at)outlook(dot)com> |
---|---|
To: | jian he <jian(dot)universality(at)gmail(dot)com>, Andrew Dunstan <andrew(at)dunslane(dot)net> |
Cc: | "pgsql-hackers(at)postgresql(dot)org" <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: [PATCH] Fix jsonb comparison for raw scalar pseudo arrays |
Date: | 2024-12-09 13:27:49 |
Message-ID: | OSBPR01MB451905E6F431F7D68515A0C48E3C2@OSBPR01MB4519.jpnprd01.prod.outlook.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Sorry, I uploaded the wrong file. I uploaded a new patch with the modified document. Please take a review. Thanks!
From: Yan Chengpeng <chengpeng_yan(at)outlook(dot)com>
Date: Monday, December 9, 2024 at 21:22
To: jian he <jian(dot)universality(at)gmail(dot)com>, Andrew Dunstan <andrew(at)dunslane(dot)net>
Cc: pgsql-hackers(at)postgresql(dot)org <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: [PATCH] Fix jsonb comparison for raw scalar pseudo arrays
If many people are already using this ‘wrong’ behavior. I agree to change the doc. I also think using ‘null’ may be a better choice. Thanks for your comments.
From: jian he <jian(dot)universality(at)gmail(dot)com>
Date: Monday, December 9, 2024 at 16:56
To: Andrew Dunstan <andrew(at)dunslane(dot)net>
Cc: Yan Chengpeng <chengpeng_yan(at)outlook(dot)com>, pgsql-hackers(at)postgresql(dot)org <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: [PATCH] Fix jsonb comparison for raw scalar pseudo arrays
On Sun, Dec 8, 2024 at 10:58 PM Andrew Dunstan <andrew(at)dunslane(dot)net> wrote:
>
> So the actual sort order as implemented is, AIUI,
>
> Object > Non-Empty-Array > Boolean > Number > String > Null > Empty-Array
>
> which is ugly, but fortunately not many apps rely on jsonb sort order.
>
> Nobody else has commented, so I propose to apply this patch documenting the anomaly.
>
while at it. we can fix the appearance of jsonb null.
since
select jsonb 'Null';
select jsonb 'NULL';
will fail.
so maybe change
<replaceable>Null</replaceable> in <synopsis> section and
<replaceable>NULL</replaceable>
to
<replaceable>null</replaceable>
Attachment | Content-Type | Size |
---|---|---|
json-doc-update.patch | application/octet-stream | 1.0 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Thomas Munro | 2024-12-09 13:29:09 | Re: Giving the shared catalogues a defined encoding |
Previous Message | Greg Sabino Mullane | 2024-12-09 13:26:05 | Re: Add a warning message when using unencrypted passwords |