]> granicus.if.org Git - postgresql/commit
Minor improvements for the multivariate MCV lists
authorTomas Vondra <tomas.vondra@postgresql.org>
Wed, 27 Mar 2019 19:07:41 +0000 (20:07 +0100)
committerTomas Vondra <tomas.vondra@postgresql.org>
Wed, 27 Mar 2019 19:07:41 +0000 (20:07 +0100)
commita63b29a1dea0ce4e95f682f9d0b36994f2fcf43e
tree2ae750da45084d063f496094ec4f2942d6a211a2
parent7300a699502fe5432b05fbc75baca534b080bebb
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.
src/backend/statistics/mcv.c
src/include/statistics/statistics.h