NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commita4d357b

committed

Make HEAP_LOCK/HEAP2_LOCK_UPDATED replay reset HEAP_XMAX_INVALID.

0ac5ad5 started to compress infomask bits in WAL records. Unfortunatelythe replay routines for XLOG_HEAP_LOCK/XLOG_HEAP2_LOCK_UPDATED forgot toreset the HEAP_XMAX_INVALID (and some other) hint bits.Luckily that's not problematic in the majority of cases, because after acrash/on a standby row locks aren't meaningful. Unfortunately that doesnot hold true in the presence of prepared transactions. This means thatafter a crash, or after promotion, row level locks held by a prepared,but not yet committed, prepared transaction might not be enforced.Discussion: 20160715192319.ubfuzim4zv3rqnxv@alap3.anarazel.deBackpatch: 9.3, the oldest branch on which0ac5ad5 is present.

1 parent45639a0 commita4d357bCopy full SHA for a4d357b

File tree

1 file changed

-0

lines changed

src/backend/access/heap
- heapam.c

1 file changed

-0

lines changed

`‎src/backend/access/heap/heapam.c`

Lines changed: 4 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -8687,6 +8687,8 @@ heap_xlog_lock(XLogReaderState *record)`
`8687`	`8687`
`8688`	`8688`	`htup= (HeapTupleHeader)PageGetItem(page,lp);`
`8689`	`8689`
	`8690`	`+htup->t_infomask &= ~(HEAP_XMAX_BITS \|HEAP_MOVED);`
	`8691`	`+htup->t_infomask2 &= ~HEAP_KEYS_UPDATED;`
`8690`	`8692`	`fix_infomask_from_infobits(xlrec->infobits_set,&htup->t_infomask,`
`8691`	`8693`	`&htup->t_infomask2);`
`8692`	`8694`
`@@ -8737,6 +8739,8 @@ heap_xlog_lock_updated(XLogReaderState *record)`
`8737`	`8739`
`8738`	`8740`	`htup= (HeapTupleHeader)PageGetItem(page,lp);`
`8739`	`8741`
	`8742`	`+htup->t_infomask &= ~(HEAP_XMAX_BITS \|HEAP_MOVED);`
	`8743`	`+htup->t_infomask2 &= ~HEAP_KEYS_UPDATED;`
`8740`	`8744`	`fix_infomask_from_infobits(xlrec->infobits_set,&htup->t_infomask,`
`8741`	`8745`	`&htup->t_infomask2);`
`8742`	`8746`	`HeapTupleHeaderSetXmax(htup,xlrec->xmax);`

0 commit comments

Comments

(0)

Movatterモバイル変換

Navigation Menu

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

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Commita4d357b

File tree

1 file changed

1 file changed

`‎src/backend/access/heap/heapam.c`

0 commit comments