NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commit87eb255

committed

Backpatch tuplesort.c assertion.

Backpatch an assertion (that was originally added to Postgres 12 bycommitdd299df) that seems broadly useful. The assertion can detectviolations of the HOT invariant (i.e. no two index tuples can point tothe same heap TID) when CREATE INDEX somehow incorrectly allows that totake place.For example, a IndexBuildHeapScan/heapam_index_build_range_scan bugmight result in two tuples that both point to the same heap TID. Ifthese two tuples also happen to be duplicates, the assertion will fail.Discussion:https://postgr.es/m/CAH2-WzmBxu4o=pMsniur+bwHqCGCmV_AOLkuK6BuU7ngA6evqw@mail.gmail.comBackpatch: 9.5-11 only

1 parentbaecd27 commit87eb255Copy full SHA for 87eb255

File tree

1 file changed

-0

lines changed

src/backend/utils/sort
- tuplesort.c

1 file changed

-0

lines changed

`‎src/backend/utils/sort/tuplesort.c`

Lines changed: 6 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -4076,6 +4076,9 @@ comparetup_index_btree(const SortTuple a, const SortTuple b,`
`4076`	`4076`	`return (pos1<pos2) ?-1 :1;`
`4077`	`4077`	`}`
`4078`	`4078`
	`4079`	`+/* ItemPointer values should never be equal */`
	`4080`	`+Assert(false);`
	`4081`	`+`
`4079`	`4082`	`return0;`
`4080`	`4083`	`}`
`4081`	`4084`
`@@ -4128,6 +4131,9 @@ comparetup_index_hash(const SortTuple a, const SortTuple b,`
`4128`	`4131`	`return (pos1<pos2) ?-1 :1;`
`4129`	`4132`	`}`
`4130`	`4133`
	`4134`	`+/* ItemPointer values should never be equal */`
	`4135`	`+Assert(false);`
	`4136`	`+`
`4131`	`4137`	`return0;`
`4132`	`4138`	`}`
`4133`	`4139`

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!

Commit87eb255

File tree

1 file changed

1 file changed

`‎src/backend/utils/sort/tuplesort.c`

0 commit comments