pgsql: Prevent GIN deleted pages from being reclaimed too early - Mailing list pgsql-committers

From Alexander Korotkov
Subject pgsql: Prevent GIN deleted pages from being reclaimed too early
Date
Msg-id E1gXI6u-0005TZ-8U@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Prevent GIN deleted pages from being reclaimed too early

When GIN vacuum deletes a posting tree page, it assumes that no concurrent
searchers can access it, thanks to ginStepRight() locking two pages at once.
However, since 9.4 searches can skip parts of posting trees descending from the
root.  That leads to the risk that page is deleted and reclaimed before
concurrent search can access it.

This commit prevents the risk of above by waiting for every transaction, which
might wait to reference this page, to finish.  Due to binary compatibility
we can't change GinPageOpaqueData to store corresponding transaction id.
Instead we reuse page header pd_prune_xid field, which is unused in index pages.

Discussion: https://postgr.es/m/31a702a.14dd.166c1366ac1.Coremail.chjischj%40163.com
Author: Andrey Borodin, Alexander Korotkov
Reviewed-by: Alexander Korotkov
Backpatch-through: 9.4

Branch
------
REL9_4_STABLE

Details
-------
https://git.postgresql.org/pg/commitdiff/1cf175c74f103e9f0905c41d761940282c3d21f1

Modified Files
--------------
src/backend/access/gin/README      | 10 ++++------
src/backend/access/gin/ginutil.c   |  7 +------
src/backend/access/gin/ginvacuum.c |  6 +++++-
src/include/access/gin_private.h   | 12 ++++++++++++
4 files changed, 22 insertions(+), 13 deletions(-)


pgsql-committers by date:

Previous
From: Alexander Korotkov
Date:
Subject: pgsql: Prevent deadlock in ginRedoDeletePage()
Next
From: Alexander Korotkov
Date:
Subject: pgsql: Fix deadlock in GIN vacuum introduced by 218f51584d5