pgsql: Fix pg_dump for disabled triggers on partitioned tables - Mailing list pgsql-committers

From Alvaro Herrera
Subject pgsql: Fix pg_dump for disabled triggers on partitioned tables
Date
Msg-id E1m4Vj5-0001Sw-0j@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Fix pg_dump for disabled triggers on partitioned tables

pg_dump failed to preserve the 'enabled' flag (which can be not only
disabled, but also REPLICA or ALWAYS) for partitions which had it
changed from their respective parents.  Attempt to handle that by
including a definition for such triggers in the dump, but replace the
standard CREATE TRIGGER line with an ALTER TRIGGER line.

Backpatch to 11, where these triggers can exist.  In branches 11 and 12,
pick up a few test lines from commit b9b408c48724 to verify that
pg_upgrade is okay with these arrangements.

Co-authored-by: Justin Pryzby <pryzby@telsasoft.com>
Co-authored-by: Álvaro Herrera <alvherre@alvh.no-ip.org>
Discussion: https://postgr.es/m/20200930223450.GA14848@telsasoft.com

Branch
------
REL_12_STABLE

Details
-------
https://git.postgresql.org/pg/commitdiff/5992c94dc7e4782760094a13be632a39ba4beffd

Modified Files
--------------
src/bin/pg_dump/pg_dump.c                  | 94 ++++++++++++++++++++++++++++--
src/bin/pg_dump/pg_dump.h                  |  1 +
src/bin/pg_dump/t/002_pg_dump.pl           | 73 ++++++++++++++++++++---
src/test/regress/expected/sanity_check.out |  5 ++
src/test/regress/expected/triggers.out     | 14 +++++
src/test/regress/sql/triggers.sql          | 15 +++++
6 files changed, 189 insertions(+), 13 deletions(-)


pgsql-committers by date:

Previous
From: Alvaro Herrera
Date:
Subject: pgsql: Fix pg_dump for disabled triggers on partitioned tables
Next
From: Tom Lane
Date:
Subject: pgsql: Doc: document the current-transaction-modes GUCs.