pgsql: Use column collation for extended statistics

Поиск
Список
Период
Сортировка
От Tomas Vondra
Тема pgsql: Use column collation for extended statistics
Дата
Msg-id E1hoqUa-0001ay-Sf@gemulon.postgresql.org
обсуждение исходный текст
Список pgsql-committers
Use column collation for extended statistics

The current extended statistics code was a bit confused which collation
to use.  When building the statistics, the collations defined as default
for the data types were used (since commit 5e0928005).  The MCV code was
however using the column collations for MCV serialization, and then
DEFAULT_COLLATION_OID when computing estimates. So overall the code was
using all three possible options, inconsistently.

This uses the column colation everywhere - this makes it consistent with
what 5e0928005 did for regular stats.  We however do not track the
collations in a catalog, because we can derive them from column-level
information.  This may need to change in the future, e.g. after allowing
statistics on expressions.

Reviewed-by: Tom Lane
Discussion: https://postgr.es/m/8736jdhbhc.fsf%40ansel.ydns.eu
Backpatch-to: 12

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/a63378a03ec0a53c7c579dfdb3abff57811d8ced

Modified Files
--------------
src/backend/commands/statscmds.c      |  4 ++++
src/backend/statistics/dependencies.c |  2 +-
src/backend/statistics/mcv.c          | 15 +++++++++++----
src/backend/statistics/mvdistinct.c   |  2 +-
4 files changed, 17 insertions(+), 6 deletions(-)


В списке pgsql-committers по дате отправления:

Предыдущее
От: Tomas Vondra
Дата:
Сообщение: pgsql: Rework examine_opclause_expression to use varonleft
Следующее
От: Peter Geoghegan
Дата:
Сообщение: pgsql: Don't rely on estimates for amcheck Bloom filters.