pgsql: Justify nbtree page split locking in code comment.

From: Peter Geoghegan <pg(at)bowt(dot)ie>
To: pgsql-committers(at)lists(dot)postgresql(dot)org
Subject: pgsql: Justify nbtree page split locking in code comment.
Date: 2020-03-27 23:46:18
Message-ID: E1jHyg6-0003KF-7G@gemulon.postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Justify nbtree page split locking in code comment.

Delaying unlocking the right child page until after the point that the
left child's parent page has been refound is no longer truly necessary.
Commit 40dae7ec made nbtree tolerant of interrupted page splits. VACUUM
was taught to avoid deleting a page that happens to be the right half of
an incomplete split. As long as page splits don't unlock the left child
page until the end of the second/final phase, it should be safe to
unlock the right child page earlier (at the end of the first phase).

It probably isn't actually useful to release the right child's lock
earlier like this (it probably won't improve performance). Even still,
pointing out that it ought to be safe to do so should make it easier to
understand the overall design.

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/9945ad6e904041be1567a89916ddc64087ed1edc

Modified Files
--------------
src/backend/access/nbtree/nbtinsert.c | 13 +++++++++++--
1 file changed, 11 insertions(+), 2 deletions(-)

Browse pgsql-committers by date

  From Date Subject
Next Message David Rowley 2020-03-28 06:21:00 pgsql: Trigger autovacuum based on number of INSERTs
Previous Message Alvaro Herrera 2020-03-27 22:57:29 pgsql: Allow walreceiver configuration to change on reload