pgsql: Fix erroneous replay of GIN_UPDATE_META_PAGE WAL records.

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: pgsql-committers(at)postgresql(dot)org
Subject: pgsql: Fix erroneous replay of GIN_UPDATE_META_PAGE WAL records.
Date: 2011-11-25 19:00:02
Message-ID: E1RU10E-0000hS-W4@gemulon.postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Fix erroneous replay of GIN_UPDATE_META_PAGE WAL records.

A simple thinko in ginRedoUpdateMetapage, namely failing to increment a
loop counter, led to inserting records into the last pending-list page in
the wrong order (the opposite of that intended). So far as I can tell,
this would not upset the code that eventually flushes pending items into
the main part of the GIN index. But it did break the code that searched
the pending list for matches, resulting in transient failure to find
matching entries during index lookups, as illustrated in bug #6307 from
Maksym Boguk.

Back-patch to 8.4 where the incorrect code was introduced.

Branch
------
master

Details
-------
http://git.postgresql.org/pg/commitdiff/877b67c38b946dcbf70fe11736bdde841e4c826b

Modified Files
--------------
src/backend/access/gin/ginxlog.c | 2 ++
1 files changed, 2 insertions(+), 0 deletions(-)

Browse pgsql-committers by date

  From Date Subject
Next Message Tom Lane 2011-11-25 19:00:03 pgsql: Fix erroneous replay of GIN_UPDATE_META_PAGE WAL records.
Previous Message Dean Rasheed 2011-11-25 16:45:34 Re: [COMMITTERS] pgsql: Enable CHECK constraints to be declared NOT VALID