Re: [PATCH] Cover get_json_table_plan() with tests - Mailing list pgsql-hackers

From Aleksander Alekseev
Subject Re: [PATCH] Cover get_json_table_plan() with tests
Date
Msg-id CAJ7c6TNzXN2WxsHQB91D4_26ds+7yK5CpV694ib--ZzJsQa9vQ@mail.gmail.com
Whole thread
In response to Re: [PATCH] Cover get_json_table_plan() with tests  (solai v <solai.cdac@gmail.com>)
Responses Re: [PATCH] Cover get_json_table_plan() with tests
List pgsql-hackers
Hi Solai,

Thanks for the feedback!

> I tested this patch on the current PostgreSQL master.The patch applied
> cleanly, but make check reported one regression test failure in
> sqljson_jsontable.
> Looking at regression.diffs, the failure comes from the newly added
> JSON_TABLE ... PLAN (...) statements. On my setup, they fail with:
> ERROR: syntax error at or near "PLAN"
> I also tried running the same JSON_TABLE ... PLAN (...) query manually
> in psql and got the same syntax error.
> Apart from that, I verified that the existing JSON_TABLE functionality
> still works as expected. I tested SELECT, EXPLAIN, EXPLAIN VERBOSE,
> and EXPLAIN ANALYZE, and they all produced the expected results.
> Is this patch expected to be applied on top of another patch that adds
> support for the JSON_TABLE PLAN (...) syntax?

The patch needs a rebase after today's commits. I will figure it out
and submit the updated version.

-- 
Best regards,
Aleksander Alekseev



pgsql-hackers by date:

Previous
From: Daniel Gustafsson
Date:
Subject: Re: Fix GROUP BY ALL handling of ORDER BY operator semantics
Next
From: Peter Smith
Date:
Subject: Re: Support EXCEPT for TABLES IN SCHEMA publications