diff options
| author | Noah Misch <noah@leadboat.com> | 2024-07-13 08:09:33 -0700 |
|---|---|---|
| committer | Noah Misch <noah@leadboat.com> | 2024-07-13 08:09:33 -0700 |
| commit | 7102070329d8147246d2791321f9915c3b5abf31 (patch) | |
| tree | fa02e622bfed4ce83620abb4970eb29c4d3cf49f /src/backend/commands/analyze.c | |
| parent | 055891f374a347aedd53c5b4fc79caf0a5522e18 (diff) | |
| download | postgresql-7102070329d8147246d2791321f9915c3b5abf31.tar.gz postgresql-7102070329d8147246d2791321f9915c3b5abf31.zip | |
Don't lose partitioned table reltuples=0 after relhassubclass=f.
ANALYZE sets relhassubclass=f when a partitioned table no longer has
partitions. An ANALYZE doing that proceeded to apply the inplace update
of pg_class.reltuples to the old pg_class tuple instead of the new
tuple, losing that reltuples=0 change if the ANALYZE committed.
Non-partitioning inheritance trees were unaffected. Back-patch to v14,
where commit 375aed36ad83f0e021e9bdd3a0034c0c992c66dc introduced
maintenance of partitioned table pg_class.reltuples.
Reported by Alexander Lakhin.
Discussion: https://postgr.es/m/a295b499-dcab-6a99-c06e-01cf60593344@gmail.com
Diffstat (limited to 'src/backend/commands/analyze.c')
| -rw-r--r-- | src/backend/commands/analyze.c | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/src/backend/commands/analyze.c b/src/backend/commands/analyze.c index 7d2cd249972..c590a2adc35 100644 --- a/src/backend/commands/analyze.c +++ b/src/backend/commands/analyze.c @@ -629,7 +629,11 @@ do_analyze_rel(Relation onerel, VacuumParams *params, else relallvisible = 0; - /* Update pg_class for table relation */ + /* + * Update pg_class for table relation. CCI first, in case acquirefunc + * updated pg_class. + */ + CommandCounterIncrement(); vac_update_relstats(onerel, relpages, totalrows, @@ -664,6 +668,7 @@ do_analyze_rel(Relation onerel, VacuumParams *params, * Partitioned tables don't have storage, so we don't set any fields * in their pg_class entries except for reltuples and relhasindex. */ + CommandCounterIncrement(); vac_update_relstats(onerel, -1, totalrows, 0, hasindex, InvalidTransactionId, InvalidMultiXactId, |