pgsql: Ensure stored generated columns must be published when required.

From: Amit Kapila <akapila(at)postgresql(dot)org>
To: pgsql-committers(at)lists(dot)postgresql(dot)org
Subject: pgsql: Ensure stored generated columns must be published when required.
Date: 2024-12-04 04:27:48
Message-ID: [email protected]
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Ensure stored generated columns must be published when required.

Ensure stored generated columns that are part of REPLICA IDENTITY must be
published explicitly for UPDATE and DELETE operations to be published. We
can publish generated columns by listing them in the column list or by
enabling the publish_generated_columns option.

This commit changes the behavior of the test added in commit adedf54e65 by
giving an ERROR for the UPDATE operation in such cases. There is no way to
trigger the bug reported in commit adedf54e65 but we didn't remove the
corresponding code change because it is still relevant when replicating
changes from a publisher with version less than 18.

We decided not to backpatch this behavior change to avoid the risk of
breaking existing output plugins that may be sending generated columns by
default although we are not aware of any such plugin. Also, we didn't see
any reports related to this on STABLE branches which is another reason not
to backpatch this change.

Author: Shlok Kyal, Hou Zhijie
Reviewed-by: Vignesh C, Amit Kapila
Discussion: https://postgr.es/m/CANhcyEVw4V2Awe2AB6i0E5AJLNdASShGfdBLbUd1XtWDboymCA@mail.gmail.com

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/87ce27de6963091f4a365f80bcdb06b9da098f00

Modified Files
--------------
doc/src/sgml/ref/create_publication.sgml | 8 ++
src/backend/commands/publicationcmds.c | 156 +++++++++++++++++-------------
src/backend/executor/execReplication.c | 39 ++++++--
src/backend/utils/cache/relcache.c | 57 ++++++++---
src/include/catalog/pg_publication.h | 7 ++
src/include/commands/publicationcmds.h | 7 +-
src/test/regress/expected/publication.out | 26 +++++
src/test/regress/sql/publication.sql | 27 ++++++
src/test/subscription/t/100_bugs.pl | 16 +--
9 files changed, 243 insertions(+), 100 deletions(-)

Browse pgsql-committers by date

  From Date Subject
Next Message Peter Eisentraut 2024-12-04 07:41:33 pgsql: Simplify IsIndexUsableForReplicaIdentityFull()
Previous Message Bruce Momjian 2024-12-04 03:31:35 pgsql: Properly use $(AWK) in Makefile, not 'awk'