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(-)
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 |