Re: [PATCH] Cover get_json_table_plan() with tests

From: Aleksander Alekseev <aleksander(at)tigerdata(dot)com>
To: PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Cc: "Jonathan Gonzalez V(dot)" <jonathan(dot)abdiel(at)gmail(dot)com>, solai v <solai(dot)cdac(at)gmail(dot)com>
Subject: Re: [PATCH] Cover get_json_table_plan() with tests
Date: 2026-07-21 22:49:34
Message-ID: CAJ7c6TPPN2rCsPxHuxc_WYbAYX0=MVs9Mq66KhA5BDzwH5pcWQ@mail.gmail.com
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Hi Jonathan,

Thanks for the feedback.

> I checked on the coverage that this patch effectively improves, but
> reviewing the report I notice that there's still some gap that can be
> fixed by just adding another nested path like this:
>
> NESTED PATH '$[*]' AS p1 COLUMNS (NESTED PATH '$[*]' AS p10 COLUMNS (a int))
> NESTED PATH '$[*]' AS p2 COLUMNS (NESTED PATH '$[*]' AS p20 COLUMNS (b int))

Could you please clarify which lines of get_json_table_plan() this
will cover which are not currently covered by patch v2?

--
Best regards,
Aleksander Alekseev

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Michael Paquier 2026-07-21 23:18:44 Re: pgstat: add pgstat_prep_pending() for entry ref pending setup
Previous Message Michael Paquier 2026-07-21 22:38:03 Re: [PATCH] Fix memory leak in pg_config