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:12:58 +0200 |
commit | 02f377dbe50074a7016f4a476a8d15d294cd0874 (patch) | |
tree | 8e768e1a7bde8cc0fac1b4a8f35d2b3118d061bc /src/backend/access/gist/gistxlog.c | |
parent | ef007e67022bf7f2367aa10fd226d6fb86b6fb9c (diff) | |
download | postgresql-02f377dbe50074a7016f4a476a8d15d294cd0874.tar.gz postgresql-02f377dbe50074a7016f4a476a8d15d294cd0874.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/access/gist/gistxlog.c')
0 files changed, 0 insertions, 0 deletions