| From: | Dmitry Dolgov <9erthalion6(at)gmail(dot)com> |
|---|---|
| To: | Dian M Fay <dian(dot)m(dot)fay(at)gmail(dot)com> |
| Cc: | Pavel Stehule <pavel(dot)stehule(at)gmail(dot)com>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Andres Freund <andres(at)anarazel(dot)de>, Alexander Korotkov <aekorotkov(at)gmail(dot)com>, Justin Pryzby <pryzby(at)telsasoft(dot)com>, David Steele <david(at)pgmasters(dot)net>, Nikita Glukhov <n(dot)gluhov(at)postgrespro(dot)ru>, Alvaro Herrera <alvherre(at)2ndquadrant(dot)com>, David Fetter <david(at)fetter(dot)org>, Thomas Munro <thomas(dot)munro(at)gmail(dot)com>, Oleksandr Shulgin <oleksandr(dot)shulgin(at)zalando(dot)de>, Robert Haas <robertmhaas(at)gmail(dot)com>, Michael Paquier <michael(dot)paquier(at)gmail(dot)com>, Oleg Bartunov <obartunov(at)gmail(dot)com>, Peter Eisentraut <peter(dot)eisentraut(at)2ndquadrant(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org> |
| Subject: | Re: [HACKERS] [PATCH] Generic type subscripting |
| Date: | 2021-01-20 16:22:31 |
| Message-ID: | 20210120162231.4oht6mdp6v76xfhs@localhost |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
> On Tue Jan 19, 2021 at 1:42 PM EST, Pavel Stehule wrote:
>
> I found minor issues.
>
> Doc - missing tag
>
> and three whitespaces issues
>
> see attached patch
Thanks, I need to remember to not skipp doc building for testing process
even for such small changes. Hope now I didn't forget anything.
> On Wed, Jan 20, 2021 at 09:58:43AM -0500, Dian M Fay wrote:
> Here's a full editing pass on the documentation, with v45 and Pavel's
> doc-whitespaces-fix.patch applied. I also corrected a typo in one of the
> added hints.
Great! I've applied almost all of it, except:
+ A <type>jsonb</type> value will accept assignments to nonexistent subscript
+ paths as long as the nonexistent elements being traversed are all arrays.
Maybe I've misunderstood the intention, but there is no requirement
about arrays for creating such an empty path. I've formulated it as:
+ A <type>jsonb</type> value will accept assignments to nonexistent subscript
+ paths as long as the last existing path key is an object or an array.
| Attachment | Content-Type | Size |
|---|---|---|
| v46-0001-Subscripting-for-jsonb.patch | text/x-diff | 45.6 KB |
| v46-0002-Filling-gaps-in-jsonb.patch | text/x-diff | 21.0 KB |
| v46-0003-Replace-assuming-a-composite-object-on-a-scalar.patch | text/x-diff | 5.5 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Dian M Fay | 2021-01-20 16:34:16 | Re: [HACKERS] [PATCH] Generic type subscripting |
| Previous Message | Alvaro Herrera | 2021-01-20 15:54:50 | Re: Allow CLUSTER, VACUUM FULL and REINDEX to change tablespace on the fly |