NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commitbd9f4cf

committed

Demote pg_unreachable() in heapam to an assertion.

Commitd168b66, which overhauled index deletion, added apg_unreachable() to the end of a sort comparator used when sorting heapTIDs from an index page. This allows the compiler to applyoptimizations that assume that the heap TIDs from the index AM mustalways be unique.That doesn't seem like a good idea now, given recent reports ofcorruption involving duplicate TIDs in indexes on Postgres 14. Demoteto an assertion, just in case.Backpatch: 14-, where index deletion was overhauled.

1 parent0c8a40b commitbd9f4cfCopy full SHA for bd9f4cf

File tree

1 file changed

-1

lines changed

src/backend/access/heap
- heapam.c

1 file changed

-1

lines changed

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

Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -7579,7 +7579,7 @@ index_delete_sort_cmp(TM_IndexDelete deltid1, TM_IndexDelete deltid2)`
`7579`	`7579`	`return (pos1<pos2) ?-1 :1;`
`7580`	`7580`	`}`
`7581`	`7581`
`7582`		`-pg_unreachable();`
	`7582`	`+Assert(false);`
`7583`	`7583`
`7584`	`7584`	`return0;`
`7585`	`7585`	`}`

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!

Commitbd9f4cf

File tree

1 file changed

1 file changed

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

0 commit comments