From: | Michael Paquier <michael(at)paquier(dot)xyz> |
---|---|
To: | pgsql-committers(at)lists(dot)postgresql(dot)org |
Subject: | pgsql: Preserve index data in pg_statistic across REINDEX CONCURRENTLY |
Date: | 2020-11-01 12:26:01 |
Message-ID: | E1kZCQr-000056-5p@gemulon.postgresql.org |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers |
Preserve index data in pg_statistic across REINDEX CONCURRENTLY
Statistics associated to an index got lost after running REINDEX
CONCURRENTLY, while the non-concurrent case preserves these correctly.
The concurrent and non-concurrent operations need to be consistent for
the end-user, and missing statistics would force to wait for a new
analyze to happen, which could take some time depending on the activity
of the existing autovacuum workers. This issue is fixed by copying any
existing entries in pg_statistic associated to the old index to the new
one. Note that this copy is already done with the data of the index in
the stats collector.
Reported-by: Fabrízio de Royes Mello
Author: Michael Paquier, Fabrízio de Royes Mello
Reviewed-by: Justin Pryzby
Discussion: https://postgr.es/m/CAFcNs+qpFPmiHd1oTXvcPdvAHicJDA9qBUSujgAhUMJyUMb+SA@mail.gmail.com
Backpatch-through: 12
Branch
------
REL_13_STABLE
Details
-------
https://git.postgresql.org/pg/commitdiff/bb62df46bcaa109d5eb1907392034024dde0886e
Modified Files
--------------
src/backend/catalog/heap.c | 41 ++++++++++++++++++++++++++++++
src/backend/catalog/index.c | 3 +++
src/include/catalog/heap.h | 1 +
src/test/regress/expected/create_index.out | 22 ++++++++++++++++
src/test/regress/sql/create_index.sql | 12 +++++++++
5 files changed, 79 insertions(+)
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2020-11-01 16:27:17 | pgsql: Avoid null pointer dereference if error result lacks SQLSTATE. |
Previous Message | Michael Paquier | 2020-11-01 10:23:34 | pgsql: Add error code for encryption failure in pgcrypto |