diff options
| author | Noah Misch <noah@leadboat.com> | 2021-01-30 00:00:27 -0800 |
|---|---|---|
| committer | Noah Misch <noah@leadboat.com> | 2021-01-30 00:02:08 -0800 |
| commit | d1ab4bf6ed2d5d5026e13af510d0d3b025fa6ac9 (patch) | |
| tree | 06806818465e33354cacc3efb09102460533d050 /src/backend/storage/lmgr/lmgr.c | |
| parent | 289ef386dff6662b1c1cf90d0666a3863bcf5ba8 (diff) | |
| download | postgresql-d1ab4bf6ed2d5d5026e13af510d0d3b025fa6ac9.tar.gz postgresql-d1ab4bf6ed2d5d5026e13af510d0d3b025fa6ac9.zip | |
Fix CREATE INDEX CONCURRENTLY for simultaneous prepared transactions.
In a cluster having used CREATE INDEX CONCURRENTLY while having enabled
prepared transactions, queries that use the resulting index can silently
fail to find rows. Fix this for future CREATE INDEX CONCURRENTLY by
making it wait for prepared transactions like it waits for ordinary
transactions. This expands the VirtualTransactionId structure domain to
admit prepared transactions. It may be necessary to reindex to recover
from past occurrences. Back-patch to 9.5 (all supported versions).
Andrey Borodin, reviewed (in earlier versions) by Tom Lane and Michael
Paquier.
Discussion: https://postgr.es/m/2E712143-97F7-4890-B470-4A35142ABC82@yandex-team.ru
Diffstat (limited to 'src/backend/storage/lmgr/lmgr.c')
| -rw-r--r-- | src/backend/storage/lmgr/lmgr.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/src/backend/storage/lmgr/lmgr.c b/src/backend/storage/lmgr/lmgr.c index 044b0189e7e..a0a0a9fea90 100644 --- a/src/backend/storage/lmgr/lmgr.c +++ b/src/backend/storage/lmgr/lmgr.c @@ -846,8 +846,7 @@ WaitForLockersMultiple(List *locktags, LOCKMODE lockmode) /* * Note: GetLockConflicts() never reports our own xid, hence we need not - * check for that. Also, prepared xacts are not reported, which is fine - * since they certainly aren't going to do anything anymore. + * check for that. Also, prepared xacts are reported and awaited. */ /* Finally wait for each such transaction to complete */ |