]> git.ipfire.org Git - thirdparty/postgresql.git/commit
Fix incorrect NEW references to generated columns in rule rewriting
authorRichard Guo <rguo@postgresql.org>
Tue, 21 Apr 2026 05:31:15 +0000 (14:31 +0900)
committerRichard Guo <rguo@postgresql.org>
Tue, 21 Apr 2026 05:33:07 +0000 (14:33 +0900)
commit07b257189ea4fc949a36fd341ba2ba6b49e5f718
tree84bd9c0672cccdf83c5509b73f89c00b9b0d0894
parent2d347f2cd71364d94cb118310492cb4dc022e05f
Fix incorrect NEW references to generated columns in rule rewriting

When a rule action or rule qualification references NEW.col where col
is a generated column (stored or virtual), the rewriter produces
incorrect results.

rewriteTargetListIU removes generated columns from the query's target
list, since stored generated columns are recomputed by the executor
and virtual ones store nothing.  However, ReplaceVarsFromTargetList
then cannot find these columns when resolving NEW references during
rule rewriting.  For UPDATE, the REPLACEVARS_CHANGE_VARNO fallback
redirects NEW.col to the original target relation, making it read the
pre-update value (same as OLD.col).  For INSERT,
REPLACEVARS_SUBSTITUTE_NULL replaces it with NULL.  Both are wrong
when the generated column depends on columns being modified.

Fix by building target list entries for generated columns from their
generation expressions, pre-resolving the NEW.attribute references
within those expressions against the query's targetlist, and passing
them together with the query's targetlist to ReplaceVarsFromTargetList.

Back-patch to all supported branches.  Virtual generated columns were
added in v18, so the back-patches in pre-v18 branches only handle
stored generated columns.

Reported-by: SATYANARAYANA NARLAPURAM <satyanarlapuram@gmail.com>
Author: Richard Guo <guofenglinux@gmail.com>
Author: Dean Rasheed <dean.a.rasheed@gmail.com>
Reviewed-by: Chao Li <li.evan.chao@gmail.com>
Discussion: https://postgr.es/m/CAHg+QDexGTmCZzx=73gXkY2ZADS6LRhpnU+-8Y_QmrdTS6yUhA@mail.gmail.com
Backpatch-through: 14
src/backend/rewrite/rewriteHandler.c
src/test/regress/expected/generated.out
src/test/regress/sql/generated.sql