Re: SQL/JSON: JSON_TABLE

From: Erik Rijkers <er(at)xs4all(dot)nl>
To: Andrew Dunstan <andrew(dot)dunstan(at)2ndquadrant(dot)com>, Andrew Dunstan <andrew(at)dunslane(dot)net>, Nikita Glukhov <n(dot)gluhov(at)postgrespro(dot)ru>, David Steele <david(at)pgmasters(dot)net>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>
Cc: Michael Paquier <michael(at)paquier(dot)xyz>, Justin Pryzby <pryzby(at)telsasoft(dot)com>, Pavel Stehule <pavel(dot)stehule(at)gmail(dot)com>, Robert Haas <robertmhaas(at)gmail(dot)com>, Dmitry Dolgov <9erthalion6(at)gmail(dot)com>, Oleg Bartunov <obartunov(at)gmail(dot)com>, Zhihong Yu <zyu(at)yugabyte(dot)com>
Subject: Re: SQL/JSON: JSON_TABLE
Date: 2021-07-10 09:07:18
Message-ID: e1cb24f2-7c11-8916-5b62-f029fb3225b1@xs4all.nl
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On 5/18/21 9:23 PM, Andrew Dunstan wrote:
>
> On 5/8/21 2:23 PM, Andrew Dunstan wrote:
>> On 4/12/21 11:34 AM, Erik Rijkers wrote:
>>>> On 2021.03.27. 02:12 Nikita Glukhov <n(dot)gluhov(at)postgrespro(dot)ru> wrote:
>>>> Attached 47th version of the patches.
>>> We're past feature freeze for 14 and alas, JSON_TABLE has not made it.
>>>
>>> I have tested quite a bit with it and because I didn't find any trouble with functionality or speed, I wanted to at least mention that here once.
>>>
>>> I looked at v47, these files
>>>> [0001-SQL-JSON-functions-v47.patch]
>>>> [0002-JSON_TABLE-v47.patch]
>>>> [0003-JSON_TABLE-PLAN-DEFAULT-clause-v47.patch]
>>>> [0004-JSON_TABLE-PLAN-clause-v47.patch]
>>>> [manual_addition_fixed.patch] # for this see [1], [2]
>>> (v47 doesn't apply anymore, as cfbot shows, but instances can still be built on top of 6131ffc43ff from 30 march 2021)
>>>
>>> I hope it will fare better next round, version 15.
>>
>> Me too. Here's a set that should remove the bitrot.
>
> Rebased for removal of serial schedule

Can one of you please add or integrate this patch to the JSON_TABLE changes?

It contains the fix for a bug that I reported earlier (on 2021-03-30 see
[1]). Nikita did diagnose this fix but today I noticed it was still not
included in the latest version, v49.

Thanks,

Erik Rijkers

[1]
https://www.postgresql.org/message-id/2101814418.20240.1617123418368%40webmailclassic.xs4all.nl

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

Attachment Content-Type Size
manual_addition_fixed.patch text/x-patch 396 bytes

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Fabien COELHO 2021-07-10 09:36:13 Re: pgbench logging broken by time logic changes
Previous Message Fabien COELHO 2021-07-10 08:25:22 Re: pgbench logging broken by time logic changes