Unpin buffer before inplace update waits for an XID to end.
authorNoah Misch <noah@leadboat.com>
Tue, 29 Oct 2024 16:39:55 +0000 (09:39 -0700)
committerNoah Misch <noah@leadboat.com>
Tue, 29 Oct 2024 16:39:59 +0000 (09:39 -0700)
commit0fe002d0c9a5b04bb676e9cb8f80325f7129b5f3
tree5dafc615e6f2b60affdfa8e87c68e997d0ef8c79
parent74f70cb8633d186928fad24ff60d922613f89c6d
Unpin buffer before inplace update waits for an XID to end.

Commit a07e03fd8fa7daf4d1356f7cb501ffe784ea6257 changed inplace updates
to wait for heap_update() commands like GRANT TABLE and GRANT DATABASE.
By keeping the pin during that wait, a sequence of autovacuum workers
and an uncommitted GRANT starved one foreground LockBufferForCleanup()
for six minutes, on buildfarm member sarus.  Prevent, at the cost of a
bit of complexity.  Back-patch to v12, like the earlier commit.  That
commit and heap_inplace_lock() have not yet appeared in any release.

Discussion: http://postgr.es/m/20241026184936.ae.nmisch@google.com
src/backend/access/heap/heapam.c
src/backend/access/index/genam.c
src/include/access/heapam.h