pgsql: Fix pg_dump --inserts mode for generated columns with dropped co

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: pgsql-committers(at)lists(dot)postgresql(dot)org
Subject: pgsql: Fix pg_dump --inserts mode for generated columns with dropped co
Date: 2021-11-22 20:26:05
Message-ID: E1mpFt7-0000mp-8c@gemulon.postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Fix pg_dump --inserts mode for generated columns with dropped columns.

If a table contains a generated column that's preceded by a dropped
column, dumpTableData_insert failed to account for the dropped
column, and would emit DEFAULT placeholder(s) in the wrong column(s).
This resulted in failures at restore time. The default COPY code path
did not have this bug, likely explaining why it wasn't noticed sooner.

While we're fixing this, we can be a little smarter about the
situation: (1) avoid unnecessarily fetching the values of generated
columns, (2) omit generated columns from the output, too, if we're
using --column-inserts. While these modes aren't expected to be
as high-performance as the COPY path, we might as well be as
efficient as we can; it doesn't add much complexity.

Per report from Дмитрий Иванов.
Back-patch to v12 where generated columns came in.

Discussion: https://postgr.es/m/CAPL5KHrkBniyQt5e1rafm5DdXvbgiiqfEQEJ9GjtVzN71Jj5pA@mail.gmail.com

Branch
------
REL_14_STABLE

Details
-------
https://git.postgresql.org/pg/commitdiff/aedc4600d84dd1ebd9386e72ac78f329b8be4804

Modified Files
--------------
src/bin/pg_dump/pg_dump.c | 51 ++++++++++++++++++++++++-----
src/bin/pg_dump/t/002_pg_dump.pl | 69 +++++++++++++++++++++++++++++++++++++++-
2 files changed, 111 insertions(+), 9 deletions(-)

Browse pgsql-committers by date

  From Date Subject
Next Message Tom Lane 2021-11-22 20:58:37 pgsql: Pacify perlcritic.
Previous Message Tom Lane 2021-11-22 17:55:29 pgsql: Probe $PROVE not $PERL while checking for modules needed by TAP