mirror of
https://git.postgresql.org/git/postgresql.git
synced 2024-09-06 23:29:20 +02:00
Don't build extended statistics on inheritance trees
When performing ANALYZE on inheritance trees, we collect two samples for each relation - one for the relation alone, and one for the inheritance subtree (relation and its child relations). And then we build statistics on each sample, so for each relation we get two sets of statistics. For regular (per-column) statistics this works fine, because the catalog includes a flag differentiating statistics built from those two samples. But we don't have such flag in the extended statistics catalogs, and we ended up updating the same row twice, triggering this error: ERROR: tuple already updated by self The simplest solution is to disable extended statistics on inheritance trees, which is what this commit is doing. In the future we may need to do something similar to per-column statistics, but that requires adding a flag to the catalog - and that's not backpatchable. Moreover, the current selectivity estimation code only works with individual relations, so building statistics on inheritance trees would be pointless anyway. Author: Tomas Vondra Backpatch-to: 10- Discussion: https://postgr.es/m/20190618231233.GA27470@telsasoft.com Reported-by: Justin Pryzby
This commit is contained in:
parent
d933816c04
commit
e1947f6c3e
@ -573,9 +573,15 @@ do_analyze_rel(Relation onerel, VacuumParams *params,
|
|||||||
thisdata->attr_cnt, thisdata->vacattrstats);
|
thisdata->attr_cnt, thisdata->vacattrstats);
|
||||||
}
|
}
|
||||||
|
|
||||||
/* Build extended statistics (if there are any). */
|
/*
|
||||||
BuildRelationExtStatistics(onerel, totalrows, numrows, rows, attr_cnt,
|
* Build extended statistics (if there are any).
|
||||||
vacattrstats);
|
*
|
||||||
|
* For now we only build extended statistics on individual relations,
|
||||||
|
* not for relations representing inheritance trees.
|
||||||
|
*/
|
||||||
|
if (!inh)
|
||||||
|
BuildRelationExtStatistics(onerel, totalrows, numrows, rows,
|
||||||
|
attr_cnt, vacattrstats);
|
||||||
}
|
}
|
||||||
|
|
||||||
/*
|
/*
|
||||||
|
@ -103,6 +103,14 @@ ANALYZE ab1 (a);
|
|||||||
WARNING: statistics object "public.ab1_a_b_stats" could not be computed for relation "public.ab1"
|
WARNING: statistics object "public.ab1_a_b_stats" could not be computed for relation "public.ab1"
|
||||||
ANALYZE ab1;
|
ANALYZE ab1;
|
||||||
DROP TABLE ab1;
|
DROP TABLE ab1;
|
||||||
|
-- Ensure we can build statistics for tables with inheritance.
|
||||||
|
CREATE TABLE ab1 (a INTEGER, b INTEGER);
|
||||||
|
CREATE TABLE ab1c () INHERITS (ab1);
|
||||||
|
INSERT INTO ab1 VALUES (1,1);
|
||||||
|
CREATE STATISTICS ab1_a_b_stats ON a, b FROM ab1;
|
||||||
|
ANALYZE ab1;
|
||||||
|
DROP TABLE ab1 CASCADE;
|
||||||
|
NOTICE: drop cascades to table ab1c
|
||||||
-- Verify supported object types for extended statistics
|
-- Verify supported object types for extended statistics
|
||||||
CREATE schema tststats;
|
CREATE schema tststats;
|
||||||
CREATE TABLE tststats.t (a int, b int, c text);
|
CREATE TABLE tststats.t (a int, b int, c text);
|
||||||
|
@ -73,6 +73,14 @@ ANALYZE ab1 (a);
|
|||||||
ANALYZE ab1;
|
ANALYZE ab1;
|
||||||
DROP TABLE ab1;
|
DROP TABLE ab1;
|
||||||
|
|
||||||
|
-- Ensure we can build statistics for tables with inheritance.
|
||||||
|
CREATE TABLE ab1 (a INTEGER, b INTEGER);
|
||||||
|
CREATE TABLE ab1c () INHERITS (ab1);
|
||||||
|
INSERT INTO ab1 VALUES (1,1);
|
||||||
|
CREATE STATISTICS ab1_a_b_stats ON a, b FROM ab1;
|
||||||
|
ANALYZE ab1;
|
||||||
|
DROP TABLE ab1 CASCADE;
|
||||||
|
|
||||||
-- Verify supported object types for extended statistics
|
-- Verify supported object types for extended statistics
|
||||||
CREATE schema tststats;
|
CREATE schema tststats;
|
||||||
|
|
||||||
|
Loading…
Reference in New Issue
Block a user