pgsql: Fix usage of "tableoid" in GENERATED expressions. - Mailing list pgsql-committers

From Tom Lane
Subject pgsql: Fix usage of "tableoid" in GENERATED expressions.
Date
Msg-id E1lkAPc-0004Ov-Ag@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Fix usage of "tableoid" in GENERATED expressions.

We consider this supported (though I've got my doubts that it's a
good idea, because tableoid is not immutable).  However, several
code paths failed to fill the field in soon enough, causing such
a GENERATED expression to see zero or the wrong value.  This
occurred when ALTER TABLE adds a new GENERATED column to a table
with existing rows, and during regular INSERT or UPDATE on a
foreign table with GENERATED columns.

Noted during investigation of a report from Vitaly Ustinov.
Back-patch to v12 where GENERATED came in.

Discussion: https://postgr.es/m/CAM_DEiWR2DPT6U4xb-Ehigozzd3n3G37ZB1+867zbsEVtYoJww@mail.gmail.com

Branch
------
REL_13_STABLE

Details
-------
https://git.postgresql.org/pg/commitdiff/77e3204ecbf15ab5dfd295bbc66eeeec4d9ade19

Modified Files
--------------
src/backend/commands/tablecmds.c        | 13 ++++++++-----
src/backend/executor/nodeModifyTable.c  | 24 ++++++++++++++++++------
src/test/regress/expected/generated.out | 13 ++++++++-----
src/test/regress/sql/generated.sql      |  5 ++++-
4 files changed, 38 insertions(+), 17 deletions(-)


pgsql-committers by date:

Previous
From: Tom Lane
Date:
Subject: pgsql: Restore the portal-level snapshot after procedure COMMIT/ROLLBAC
Next
From: Tom Lane
Date:
Subject: pgsql: Disallow whole-row variables in GENERATED expressions.