pgsql: Fix error reporting for SQL/JSON path type mismatches

pgsql: Fix error reporting for SQL/JSON path type mismatches

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 
Author: Jian He 
Reviewed-by: Kirill Reshke 
Discussion: https://postgr.es/m/CACJufxHunVg81JMuNo8Yvv_hJD0DicgaVN2Wteu8aJbVJPBjZA@mail.gmail.com
Backpatch-through: 17

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/519fa0433b37701b357753a568080bee2c47d238

Modified Files
--------------
src/backend/parser/parse_expr.c                  | 23 +++++++++++++++--------
src/test/regress/expected/sqljson_queryfuncs.out |  8 ++++++++
src/test/regress/sql/sqljson_queryfuncs.sql      |  2 ++
3 files changed, 25 insertions(+), 8 deletions(-)