git.postgresql.org Git - postgresql.git/commit

git projects / postgresql.git / commit
? search:
summary | shortlog | log | commit | commitdiff | tree
(parent: c2d93c3) | patch
SQL/JSON: Always coerce JsonExpr result at runtime
2024年6月28日 12:58:13 +0000 (21:58 +0900)
2024年6月28日 12:58:13 +0000 (21:58 +0900)
commit 716bd12d22c53d1943d41309f2dd061ec601dd5e
SQL/JSON: Always coerce JsonExpr result at runtime

Instead of looking up casts at parse time for converting the result
of JsonPath* query functions to the specified or the default
RETURNING type, always perform the conversion at runtime using either
the target type's input function or the function
json_populate_type().

There are two motivations for this change:

1. json_populate_type() coerces to types with typmod such that any
string values that exceed length limit cause an error instead of
silent truncation, which is necessary to be standard-conforming.

2. It was possible to end up with a cast expression that doesn't
support soft handling of errors causing bugs in the of handling
ON ERROR clause.

JsonExpr.coercion_expr which would store the cast expression is no
longer necessary, so remove.

Bump catversion because stored rules change because of the above
removal.

Reported-by: Alvaro Herrera <alvherre@alvh.no-ip.org>
Reviewed-by: Jian He <jian.universality@gmail.com>
Discussion: Discussion: https://postgr.es/m/202405271326.5a5rprki64aw%40alvherre.pgsql
15 files changed:
src/backend/executor/execExpr.c diff | blob | blame | history
src/backend/executor/execExprInterp.c diff | blob | blame | history
src/backend/jit/llvm/llvmjit_expr.c diff | blob | blame | history
src/backend/nodes/nodeFuncs.c diff | blob | blame | history
src/backend/parser/parse_expr.c diff | blob | blame | history
src/backend/utils/adt/jsonfuncs.c diff | blob | blame | history
src/include/catalog/catversion.h diff | blob | blame | history
src/include/executor/execExpr.h diff | blob | blame | history
src/include/nodes/execnodes.h diff | blob | blame | history
src/include/nodes/primnodes.h diff | blob | blame | history
src/include/utils/jsonfuncs.h diff | blob | blame | history
src/test/regress/expected/sqljson_jsontable.out diff | blob | blame | history
src/test/regress/expected/sqljson_queryfuncs.out diff | blob | blame | history
src/test/regress/sql/sqljson_jsontable.sql diff | blob | blame | history
src/test/regress/sql/sqljson_queryfuncs.sql diff | blob | blame | history
This is the main PostgreSQL git repository.
RSS Atom

AltStyle によって変換されたページ (->オリジナル) /