From: | Dean Rasheed <dean(dot)a(dot)rasheed(at)gmail(dot)com> |
---|---|
To: | pgsql-committers(at)postgresql(dot)org |
Subject: | pgsql: Improve estimate of distinct values in estimate_num_groups(). |
Date: | 2016-04-04 11:49:36 |
Message-ID: | E1an30i-0006qv-Ss@gemulon.postgresql.org |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers |
Improve estimate of distinct values in estimate_num_groups().
When adjusting the estimate for the number of distinct values from a
rel in a grouped query to take into account the selectivity of the
rel's restrictions, use a formula that is less likely to produce
under-estimates.
The old formula simply multiplied the number of distinct values in the
rel by the restriction selectivity, which would be correct if the
restrictions were fully correlated with the grouping expressions, but
can produce significant under-estimates in cases where they are not
well correlated.
The new formula is based on the random selection probability, and so
assumes that the restrictions are not correlated with the grouping
expressions. This is guaranteed to produce larger estimates, and of
course risks over-estimating in cases where the restrictions are
correlated, but that has less severe consequences than
under-estimating, which might lead to a HashAgg that consumes an
excessive amount of memory.
This could possibly be improved upon in the future by identifying
correlated restrictions and using a hybrid of the old and new
formulae.
Author: Tomas Vondra, with some hacking be me
Reviewed-by: Mark Dilger, Alexander Korotkov, Dean Rasheed and Tom Lane
Discussion: http://www.postgresql.org/message-id/flat/56CD0381(dot)5060502(at)2ndquadrant(dot)com
Branch
------
master
Details
-------
http://git.postgresql.org/pg/commitdiff/84f9a35e398f863c62440d3f82fc57b4fedc5d08
Modified Files
--------------
src/backend/utils/adt/selfuncs.c | 64 +++++++++++++++++++++++++++------
src/test/regress/expected/subselect.out | 25 ++++++-------
2 files changed, 64 insertions(+), 25 deletions(-)
From | Date | Subject | |
---|---|---|---|
Next Message | Teodor Sigaev | 2016-04-04 11:55:51 | pgsql: fix typo |
Previous Message | Andres Freund | 2016-04-04 07:50:48 | Re: pgsql: Avoid archiving XLOG_RUNNING_XACTS on idle server |