| commit | 69d7edb06fc69da114b075be58dd4318916500e5 | |
| author | Tom Lane <tgl@sss.pgh.pa.us> | |
| Fri, 29 Dec 2023 15:57:11 +0000 (29 10:57 -0500) | ||
| committer | Tom Lane <tgl@sss.pgh.pa.us> | |
| Fri, 29 Dec 2023 15:57:11 +0000 (29 10:57 -0500) | ||
| tree | 68deeb3662a411250d1603cd360fba1ebba1802e | treesnapshot (tar.gz zip) |
| parent | b901961be1d5191650cb4fae49e403843fc57ccc | commitdiff |
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: https://postgr.es/m/7075d3aa-3f05-44a5-b68f-47dc6a8a0550@buildkite.com
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: https://postgr.es/m/7075d3aa-3f05-44a5-b68f-47dc6a8a0550@buildkite.com
| src/bin/pg_dump/pg_dump.c | diffblobblamehistory | |
| src/bin/pg_dump/pg_dump.h | diffblobblamehistory | |
| src/bin/pg_dump/t/002_pg_dump.pl | diffblobblamehistory |