Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Commit47ad791

Browse files
committed
Fix bugs in Serializable Snapshot Isolation.
Change the way UPDATEs are handled. Instead of maintaining a chain oftuple-level locks in shared memory, copy any existing locks on the oldtuple to the new tuple at UPDATE. Any existing page-level lock needs tobe duplicated too, as a lock on the new tuple. That was neglectedpreviously.Store xmin on tuple-level predicate locks, to distinguish a lock on an oldalready-recycled tuple from a new tuple at the same physical location.Failure to distinguish them caused loops in the tuple-lock chains, asreported by YAMAMOTO Takashi. Although we don't use the chain representationof UPDATEs anymore, it seems like a good idea to store the xmin to avoidsome false positives if no other reason.CheckSingleTargetForConflictsIn now correctly handles the case where a lockthat's being held is not reflected in the local lock table. That happensif another backend acquires a lock on our behalf due to an UPDATE or a pagesplit.PredicateLockPageCombine now retains locks for the page that is beingremoved, rather than removing them. This prevents a potentially dangerousfalse-positive inconsistency where the local lock table believes that a lockis held, but it is actually not.Dan Ports and Kevin Grittner
1 parent16143d6 commit47ad791

File tree

4 files changed

+148
-291
lines changed

4 files changed

+148
-291
lines changed

‎src/backend/access/nbtree/nbtree.c

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -824,7 +824,6 @@ btvacuumpage(BTVacState *vstate, BlockNumber blkno, BlockNumber orig_blkno)
824824
if (_bt_page_recyclable(page))
825825
{
826826
/* Okay to recycle this page */
827-
Assert(!PageIsPredicateLocked(rel,blkno));
828827
RecordFreeIndexPage(rel,blkno);
829828
vstate->totFreePages++;
830829
stats->pages_deleted++;

0 commit comments

Comments
 (0)

[8]ページ先頭

©2009-2025 Movatter.jp