diff options
author | Heikki Linnakangas <heikki.linnakangas@iki.fi> | 2012-01-18 17:09:44 +0200 |
---|---|---|
committer | Heikki Linnakangas <heikki.linnakangas@iki.fi> | 2012-01-18 17:57:33 +0200 |
commit | 326b922e8b2d65257a635b5f80e5de0f15dffd3a (patch) | |
tree | d190b8a6468af0ab1dbaec083d5ae1d358439399 /src/backend/utils/adt/dbsize.c | |
parent | 2106c55ac8dacc52bf6483925529fd3ab99b94c4 (diff) | |
download | postgresql-326b922e8b2d65257a635b5f80e5de0f15dffd3a.tar.gz postgresql-326b922e8b2d65257a635b5f80e5de0f15dffd3a.zip |
Fix corner case in cleanup of transactions using SSI.
When the only remaining active transactions are READ ONLY, we do a "partial
cleanup" of committed transactions because certain types of conflicts
aren't possible anymore. For committed r/w transactions, we release the
SIREAD locks but keep the SERIALIZABLEXACT. However, for committed r/o
transactions, we can go further and release the SERIALIZABLEXACT too. The
problem was with the latter case: we were returning the SERIALIZABLEXACT to
the free list without removing it from the finished list.
The only real change in the patch is the SHMQueueDelete line, but I also
reworked some of the surrounding code to make it obvious that r/o and r/w
transactions are handled differently -- the existing code felt a bit too
clever.
Dan Ports
Diffstat (limited to 'src/backend/utils/adt/dbsize.c')
0 files changed, 0 insertions, 0 deletions