diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2009-12-29 20:11:45 +0000 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2009-12-29 20:11:45 +0000 |
| commit | 649b5ec7c8a3050a30bd6d36003ba3a681c9a198 (patch) | |
| tree | af041b646426d71d9ea1113837b5f7a9001082c9 /src/backend/catalog/heap.c | |
| parent | 84d723b6cefcf25b8c800f8aa6cf3c9538a546b4 (diff) | |
| download | postgresql-649b5ec7c8a3050a30bd6d36003ba3a681c9a198.tar.gz postgresql-649b5ec7c8a3050a30bd6d36003ba3a681c9a198.zip | |
Add the ability to store inheritance-tree statistics in pg_statistic,
and teach ANALYZE to compute such stats for tables that have subclasses.
Per my proposal of yesterday.
autovacuum still needs to be taught about running ANALYZE on parent tables
when their subclasses change, but the feature is useful even without that.
Diffstat (limited to 'src/backend/catalog/heap.c')
| -rw-r--r-- | src/backend/catalog/heap.c | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/src/backend/catalog/heap.c b/src/backend/catalog/heap.c index 0f33c525b61..8ef00f9287b 100644 --- a/src/backend/catalog/heap.c +++ b/src/backend/catalog/heap.c @@ -8,7 +8,7 @@ * * * IDENTIFICATION - * $PostgreSQL: pgsql/src/backend/catalog/heap.c,v 1.362 2009/12/24 22:09:23 momjian Exp $ + * $PostgreSQL: pgsql/src/backend/catalog/heap.c,v 1.363 2009/12/29 20:11:43 tgl Exp $ * * * INTERFACE ROUTINES @@ -2314,7 +2314,7 @@ cookConstraint(ParseState *pstate, /* * RemoveStatistics --- remove entries in pg_statistic for a rel or column * - * If attnum is zero, remove all entries for rel; else remove only the one + * If attnum is zero, remove all entries for rel; else remove only the one(s) * for that column. */ void @@ -2344,9 +2344,10 @@ RemoveStatistics(Oid relid, AttrNumber attnum) nkeys = 2; } - scan = systable_beginscan(pgstatistic, StatisticRelidAttnumIndexId, true, + scan = systable_beginscan(pgstatistic, StatisticRelidAttnumInhIndexId, true, SnapshotNow, nkeys, key); + /* we must loop even when attnum != 0, in case of inherited stats */ while (HeapTupleIsValid(tuple = systable_getnext(scan))) simple_heap_delete(pgstatistic, &tuple->t_self); |