Rename JsonIsPredicate.value_type, fix JSON backend/nodes/ infrastructure.
authorTom Lane <tgl@sss.pgh.pa.us>
Fri, 13 May 2022 15:40:01 +0000 (11:40 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Fri, 13 May 2022 15:40:08 +0000 (11:40 -0400)
commit3ab9a63cb638a1fd99475668e2da9c237495aeda
treefac93993a8a296e02fd12a4b222887f0d6b7d38e
parent2454cb07eed1ec7e68b9953b03d473d3c334ba05
Rename JsonIsPredicate.value_type, fix JSON backend/nodes/ infrastructure.

I started out with the intention to rename value_type to item_type to
avoid a collision with a typedef name that appears on some platforms.

Along the way, I noticed that the adjacent field "format" was not being
correctly handled by the backend/nodes/ infrastructure functions:
copyfuncs.c erroneously treated it as a scalar, while equalfuncs,
outfuncs, and readfuncs omitted handling it at all.  This looks like
it might be cosmetic at the moment because the field is always NULL
after parse analysis; but that's likely a bug in itself, and the code's
certainly not very future-proof.  Let's fix it while we can still do so
without forcing an initdb on beta testers.

Further study found a few other inconsistencies in the backend/nodes/
infrastructure for the recently-added JSON node types, so fix those too.

catversion bumped because of potential change in stored rules.

Discussion: https://postgr.es/m/526703.1652385613@sss.pgh.pa.us
12 files changed:
src/backend/executor/execExprInterp.c
src/backend/nodes/copyfuncs.c
src/backend/nodes/equalfuncs.c
src/backend/nodes/makefuncs.c
src/backend/nodes/outfuncs.c
src/backend/nodes/readfuncs.c
src/backend/parser/parse_expr.c
src/backend/utils/adt/ruleutils.c
src/backend/utils/misc/queryjumble.c
src/include/catalog/catversion.h
src/include/nodes/makefuncs.h
src/include/nodes/primnodes.h