NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commit78a34c6

committed

During heap rebuild, lock any TOAST index until end of transaction.

swap_relation_files() calls toast_get_valid_index() to find and lockthis index, just before swapping with the rebuilt TOAST index. Thelatter function releases the lock before returning. Potential formischief is low; a concurrent session can issue ALTER INDEX ... SET(fillfactor = ...), which is not alarming. Nonetheless, changingpg_class.relfilenode without a lock is unconventional. Back-patch to9.5 (all supported versions), because another fix needs this.Discussion:https://postgr.es/m/20191226001521.GA1772687@rfd.leadboat.com

1 parent4034868 commit78a34c6Copy full SHA for 78a34c6

File tree

2 files changed

-3

lines changed

src/backend
- access/heap
  - tuptoaster.c
- commands
  - cluster.c

2 files changed

-3

lines changed

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

Lines changed: 2 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -1447,8 +1447,8 @@ toast_get_valid_index(Oid toastoid, LOCKMODE lock)`
`1447`	`1447`	`validIndexOid=RelationGetRelid(toastidxs[validIndex]);`
`1448`	`1448`
`1449`	`1449`	`/* Close the toast relation and all its indexes */`
`1450`		`-toast_close_indexes(toastidxs,num_indexes,lock);`
`1451`		`-heap_close(toastrel,lock);`
	`1450`	`+toast_close_indexes(toastidxs,num_indexes,NoLock);`
	`1451`	`+heap_close(toastrel,NoLock);`
`1452`	`1452`
`1453`	`1453`	`returnvalidIndexOid;`
`1454`	`1454`	`}`

`‎src/backend/commands/cluster.c`

Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -1636,7 +1636,7 @@ finish_heap_swap(Oid OIDOldHeap, Oid OIDNewHeap,`
`1636`	`1636`
`1637`	`1637`	`/* Get the associated valid index to be renamed */`
`1638`	`1638`	`toastidx=toast_get_valid_index(newrel->rd_rel->reltoastrelid,`
`1639`		`-AccessShareLock);`
	`1639`	`+NoLock);`
`1640`	`1640`
`1641`	`1641`	`/* rename the toast table ... */`
`1642`	`1642`	`snprintf(NewToastName,NAMEDATALEN,"pg_toast_%u",`

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!

Commit78a34c6

File tree

2 files changed

2 files changed

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

`‎src/backend/commands/cluster.c`

0 commit comments