diff options
| author | Tomas Vondra | 2019-03-27 19:07:41 +0000 |
|---|---|---|
| committer | Tomas Vondra | 2019-03-27 19:07:41 +0000 |
| commit | a63b29a1dea0ce4e95f682f9d0b36994f2fcf43e (patch) | |
| tree | 2ae750da45084d063f496094ec4f2942d6a211a2 /src/include/statistics | |
| parent | 7300a699502fe5432b05fbc75baca534b080bebb (diff) | |
Minor improvements for the multivariate MCV lists
The MCV build should always call get_mincount_for_mcv_list(), as the
there is no other logic to decide whether the MCV list represents all
the data. So just remove the (ngroups > nitems) condition.
Also, when building MCV lists, the number of items was limited by the
statistics target (i.e. up to 10000). But when deserializing the MCV
list, a different value (8192) was used to check the input, causing
an error. Simply ensure that the same value is used in both places.
This should have been included in 7300a69950, but I forgot to include it
in that commit.
Diffstat (limited to 'src/include/statistics')
| -rw-r--r-- | src/include/statistics/statistics.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/include/statistics/statistics.h b/src/include/statistics/statistics.h index 57998810755..f421c59338d 100644 --- a/src/include/statistics/statistics.h +++ b/src/include/statistics/statistics.h @@ -82,8 +82,8 @@ typedef struct MVDependencies #define STATS_MCV_MAGIC 0xE1A651C2 /* marks serialized bytea */ #define STATS_MCV_TYPE_BASIC 1 /* basic MCV list type */ -/* max items in MCV list (mostly arbitrary number) */ -#define STATS_MCVLIST_MAX_ITEMS 8192 +/* max items in MCV list (should be equal to max default_statistics_target) */ +#define STATS_MCVLIST_MAX_ITEMS 10000 /* * Multivariate MCV (most-common value) lists |