pgsql: In pg_dump, don't dump a stats object unless dumping underlying

Started by Tom Laneover 2 years ago1 messagescomitters
Jump to latest
#1Tom Lane
tgl@sss.pgh.pa.us

In pg_dump, don't dump a stats object unless dumping underlying table.

If the underlying table isn't being dumped, it's useless to dump
an extended statistics object; it'll just cause errors at restore.
We have always applied similar policies to, say, indexes.

(When and if we get cross-table stats objects, it might be profitable
to think a little harder about what to do with them. But for now
there seems no point in considering a stats object as anything but
an appendage of its table.)

Rian McGuire and Tom Lane, per report from Rian McGuire.
Back-patch to supported branches.

Discussion: /messages/by-id/7075d3aa-3f05-44a5-b68f-47dc6a8a0550@buildkite.com

Branch
------
REL_13_STABLE

Details
-------
https://git.postgresql.org/pg/commitdiff/8d4e8b746815505c3b808ab96674ec2e140afc56

Modified Files
--------------
src/bin/pg_dump/pg_dump.c | 30 +++++++++++++++++++++++++++---
src/bin/pg_dump/pg_dump.h | 1 +
src/bin/pg_dump/t/002_pg_dump.pl | 6 +++---
3 files changed, 31 insertions(+), 6 deletions(-)