pgsql: Fix broken cleanup interlock for GIN pending list. - Mailing list pgsql-committers

From Robert Haas
Subject pgsql: Fix broken cleanup interlock for GIN pending list.
Date
Msg-id E1eFQmZ-0001nY-6B@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Fix broken cleanup interlock for GIN pending list.

The pending list must (for correctness) always be cleaned up by vacuum, and
should (for the avoidance of surprising behavior) always be cleaned up
by an explicit call to gin_clean_pending_list, but cleanup is optional
when inserting.  The old logic got this backward: cleanup was forced
if (stats == NULL), but that's going to be *false* when vacuuming and
*true* for inserts.

Masahiko Sawada, reviewed by me.

Discussion: http://postgr.es/m/CAD21AoBLUSyiYKnTYtSAbC+F=XDjiaBrOUEGK+zUXdQ8owfPKw@mail.gmail.com

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/3b2787e1f8f1eeeb6bd9565288ab210262705b56

Modified Files
--------------
src/backend/access/gin/ginfast.c   | 14 +++++++++-----
src/backend/access/gin/ginvacuum.c |  6 +++---
src/include/access/gin_private.h   |  2 +-
3 files changed, 13 insertions(+), 9 deletions(-)


pgsql-committers by date:

Previous
From: Robert Haas
Date:
Subject: pgsql: Fix typo in comment.
Next
From: Robert Haas
Date:
Subject: pgsql: Remove redundant line from Makefile.