Reduce number of commands dumpTableSchema emits for binary upgrade.
commit2fa989e6a3407b9da625e1524c8694bc028e25ba
authorTom Lane <tgl@sss.pgh.pa.us>
Mon, 29 Jul 2024 15:53:49 +0000 (29 11:53 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Mon, 29 Jul 2024 15:53:49 +0000 (29 11:53 -0400)
tree199c853e939cb69159f8442ecb51a90d58c9e5bb
parent1e020258e53c87c697615390c42a191344bbb909
Reduce number of commands dumpTableSchema emits for binary upgrade.

Avoid issuing a separate SQL UPDATE command for each column when
directly manipulating pg_attribute contents in binary upgrade mode.
With the separate updates, we triggered a relcache invalidation with
each update.  For a table with N columns, that causes O(N^2) relcache
bloat in txn_size mode because the table's newly-created relcache
entry can't be flushed till end of transaction.  Reducing the number
of commands should make it marginally faster as well as avoiding that
problem.

While at it, likewise avoid issuing a separate UPDATE on pg_constraint
for each inherited constraint.  This is less exciting, first because
inherited (non-partitioned) constraints are relatively rare, and
second because the backend has a good deal of trouble anyway with
restoring tables containing many such constraints, due to
MergeConstraintsIntoExisting being horribly inefficient.  But it seems
more consistent to do it this way here too, and it surely can't hurt.

In passing, fix one place in dumpTableSchema that failed to use ONLY
in ALTER TABLE.  That's not a live bug, but it's inconsistent.
Also avoid silently casting away const from string literals.

Per report from Justin Pryzby.  Back-patch to v17 where txn_size mode
was introduced.

Discussion: https://postgr.es/m/ZqEND4ZcTDBmcv31@pryzbyj2023
src/bin/pg_dump/pg_dump.c
src/bin/pg_dump/t/002_pg_dump.pl