Re: remaining sql/json patches

Alexander Lakhin <exclusion@gmail.com>

From: Alexander Lakhin <exclusion@gmail.com>

To: Amit Langote <amitlangote09@gmail.com>, jian he <jian.universality@gmail.com>

Cc: Alvaro Herrera <alvherre@alvh.no-ip.org>, Tomas Vondra <tomas.vondra@enterprisedb.com>, Himanshu Upadhyaya <upadhyaya.himanshu@gmail.com>, Erik Rijkers <er@xs4all.nl>, Andres Freund <andres@anarazel.de>, Andrew Dunstan <andrew@dunslane.net>, PostgreSQL-development <pgsql-hackers@postgresql.org>

Date: 2024-04-05T06:00:00Z

Lists: pgsql-hackers

Commits

Same data as JSON: GET /api/v1/messages/:b64id/commits the thread's linked commits as JSON, with link sources. API reference →

SQL/JSON: Various improvements to SQL/JSON query function docs
- ce416fadb4b6 17.0 landed
- 42de72fa7b80 18.0 landed
SQL/JSON: Fix some obsolete comments.
- 290a6d800d90 17.0 landed
- 7768b6569de9 16.4 landed
- 3a8a1f3254b2 18.0 landed
SQL/JSON: Fix issues with DEFAULT .. ON ERROR / EMPTY
- c0fc0751862d 17.0 landed
JSON_TABLE: Add support for NESTED paths and columns
- bb766cde63b4 17.0 landed
Fix JsonExpr deparsing to emit QUOTES and WRAPPER correctly
- f6a2529920cf 17.0 landed
Fix typo introduced in 6185c9737
- 2f6e78b0619a 17.0 landed
Add basic JSON_TABLE() functionality
- de3600452b61 17.0 landed
Avoid splitting errmsg string to span multiple lines
- 085e759e9da7 17.0 landed
Add SQL/JSON query functions
- 6185c9737cf4 17.0 landed
Implement various jsonpath methods
- 66ea94e8e606 17.0 cited
Add soft error handling to some expression nodes
- aaaf9449ec6b 17.0 landed
- 7fbc75b26ed8 17.0 landed
Adjust populate_record_field() to handle errors softly
- 1edb3b491bee 17.0 landed
Refactor code used by jsonpath executor to fetch variables
- faa2b953ba3b 17.0 landed
Test EXPLAIN (FORMAT JSON) ... XMLTABLE
- 752533d40fd5 17.0 landed
Simplify productions for FORMAT JSON [ ENCODING name ]
- d3fe6e90bab5 17.0 landed
Add trailing commas to enum definitions
- 611806cd726f 17.0 cited
doc: add missing <returnvalue> and whitespace
- e055b6be7ebb 17.0 landed
Add more SQL/JSON constructor functions
- 03734a7fed7d 17.0 landed
Rename a nonterminal used in SQL/JSON grammar
- 254ac5a7c31f 17.0 landed
Some refactoring to export json(b) conversion functions
- b22391a2ff7b 17.0 landed
Don't include CaseTestExpr in JsonValueExpr.formatted_expr
- 66a9003e2e3e 16.0 landed
- b6e1157e7d33 17.0 landed
Code review for commit b6e1157e7d
- 7c7412cae3ea 17.0 landed
Pass constructName to transformJsonValueExpr()
- 7825a1b01e40 16.0 landed
- 785480c9533d 17.0 landed
Unify JSON categorize type API and export for external use
- 3c152a27b063 17.0 landed
Make some indentation in gram.y consistent
- 5edf438eeb00 17.0 landed
- 01f1f789df56 16.0 landed
Allow most keywords to be used as column labels without requiring AS.
- 06a7c3154f5b 14.0 cited
Reduce size of backend scanner's tables.
- 7f380c59f800 13.0 cited
Use perfect hashing, instead of binary search, for keyword lookup.
- c64d0cd5ce24 12.0 cited

Hello Amit,

04.04.2024 15:02, Amit Langote wrote:
> Pushed after fixing these and a few other issues.  I didn't include
> the testing function you proposed in your other email.  It sounds
> useful for testing locally but will need some work before we can
> include it in the tree.
>
> I'll post the rebased 0002 tomorrow after addressing your comments.

Please look at an assertion failure:
TRAP: failed Assert("count <= tupdesc->natts"), File: "parse_relation.c", Line: 3048, PID: 1325146

triggered by the following query:
SELECT * FROM JSON_TABLE('0', '$' COLUMNS (js int PATH '$')),
   COALESCE(row(1)) AS (a int, b int);

Without JSON_TABLE() I get:
ERROR:  function return row and query-specified return row do not match
DETAIL:  Returned row contains 1 attribute, but query expects 2.

Best regards,
Alexander