]> git.ipfire.org Git - thirdparty/postgresql.git/commit
Fix bug in detecting concurrent page splits in GiST insert
authorHeikki Linnakangas <heikki.linnakangas@iki.fi>
Wed, 20 Jan 2021 09:58:03 +0000 (11:58 +0200)
committerHeikki Linnakangas <heikki.linnakangas@iki.fi>
Wed, 20 Jan 2021 09:58:27 +0000 (11:58 +0200)
commit0326635dd484fe1d5272dd0c551b78e3f779acb4
tree5d6ec9c8a7e1cb7666751a87265fbef5432f2dcf
parent5ad672fc2f6541ec5a0c533fd44d3819bff1447a
Fix bug in detecting concurrent page splits in GiST insert

In commit 9eb5607e699, I got the condition on checking for split or
deleted page wrong: I used && instead of ||. The comment correctly said
"concurrent split _or_ deletion".

As a result, GiST insertion could miss a concurrent split, and insert to
wrong page. Duncan Sands demonstrated this with a test script that did a
lot of concurrent inserts.

Backpatch to v12, where this was introduced. REINDEX is required to fix
indexes that were affected by this bug.

Backpatch-through: 12
Reported-by: Duncan Sands
Discussion: https://www.postgresql.org/message-id/a9690483-6c6c-3c82-c8ba-dc1a40848f11%40deepbluecap.com
src/backend/access/gist/gist.c