Fix error reporting for SQL/JSON path type mismatches
authorAmit Langote <amitlan@postgresql.org>
Thu, 27 Nov 2025 01:42:51 +0000 (10:42 +0900)
committerAmit Langote <amitlan@postgresql.org>
Thu, 27 Nov 2025 02:59:40 +0000 (11:59 +0900)
commit15ba0702c1ae9d46f49a6e1f80db99167d3aedf7
tree527e66de26bb5baef13d81df29536c289116406d
parentb83bcc0df180056b9374bd4239d32fea84bb46f2
Fix error reporting for SQL/JSON path type mismatches

transformJsonFuncExpr() used exprType()/exprLocation() on the
possibly coerced path expression, which could be NULL when
coercion to jsonpath failed, leading to "cache lookup failed
for type 0" errors.

Preserve the original expression node so that type and location
in the "must be of type jsonpath" error are reported correctly.
Add regression tests to cover these cases.

Reported-by: Jian He <jian.universality@gmail.com>
Author: Jian He <jian.universality@gmail.com>
Reviewed-by: Kirill Reshke <reshkekirill@gmail.com>
Discussion: https://postgr.es/m/CACJufxHunVg81JMuNo8Yvv_hJD0DicgaVN2Wteu8aJbVJPBjZA@mail.gmail.com
Backpatch-through: 17
src/backend/parser/parse_expr.c
src/test/regress/expected/sqljson_queryfuncs.out
src/test/regress/sql/sqljson_queryfuncs.sql