From: | Pavel Stehule <pavel(dot)stehule(at)gmail(dot)com> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | Erik Rijkers <er(at)xs4all(dot)nl>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
Subject: | Re: Triage on old commitfest entries - JSON_PATH |
Date: | 2021-10-04 04:06:28 |
Message-ID: | CAFj8pRDCSCAzwJ-9nYYGeZANx4jOQTzRB9GFHOSb03Vc61szYA@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
ne 3. 10. 2021 v 22:16 odesílatel Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> napsal:
> Erik Rijkers <er(at)xs4all(dot)nl> writes:
> > Op 03-10-2021 om 21:14 schreef Tom Lane:
> >> I looked at entries that are at least 10 CFs old, as indicated by
> >> the handy sort field. That's a pretty small population: 16 items
> >> out of the 317 listed in the 2021-09 CF. A quick look in recent
> >> CFs shows that it's very rare that we commit entries older than
> >> 10 CFs.
>
> > May I add one more?
>
> > SQL/JSON: JSON_TABLE started 2018 (the commitfest page shows only 4 as
> > 'Age in CFs' but that obviously can't be right)
>
> Hm. It's being actively worked on, so I wouldn't have proposed
> killing it even if its age had been shown correctly. Unless you
> think it has no hope of ever reaching committability?
>
This is a pretty important feature and a nice patch.
Unfortunately, it is a pretty complex patch - JSON_TABLE is a really
complex function, and this patch does complete implementation. I checked
this patch more times, and I think it is good. There is only one problem -
the size (there are not any problems in code, or in behaviour) . In MySQL
or MariaDB, there is a much more simple implementation, that covers maybe
10% of standard. But it is available, and people can use it. Isn't it
possible to reduce this patch to some basic functionality, and commit it
quickly, and later commit step by step all parts.
Regards
Pavel
> regards, tom lane
>
>
>
From | Date | Subject | |
---|---|---|---|
Next Message | Yura Sokolov | 2021-10-04 04:18:56 | Re: BufferAlloc: don't take two simultaneous locks |
Previous Message | Pavel Stehule | 2021-10-04 03:56:59 | Re: Triage on old commitfest entries |