diff options
author | Heikki Linnakangas <heikki.linnakangas@iki.fi> | 2012-05-14 10:22:44 +0300 |
---|---|---|
committer | Heikki Linnakangas <heikki.linnakangas@iki.fi> | 2012-05-14 10:28:55 +0300 |
commit | 9e4637bf89ef9fbc89a45dc4b421fa6740accd41 (patch) | |
tree | 1654e9f6fe882b4dd080064360c8a590f46ee5d8 /src/backend/utils/time/tqual.c | |
parent | 64f09ca38600ef0471683491f7735001b72c0f39 (diff) | |
download | postgresql-9e4637bf89ef9fbc89a45dc4b421fa6740accd41.tar.gz postgresql-9e4637bf89ef9fbc89a45dc4b421fa6740accd41.zip |
Update comments that became out-of-date with the PGXACT struct.
When the "hot" members of PGPROC were split off to separate PGXACT structs,
many PGPROC fields referred to in comments were moved to PGXACT, but the
comments were neglected in the commit. Mostly this is just a search/replace
of PGPROC with PGXACT, but the way the dummy PGPROC entries are created for
prepared transactions changed more, making some of the comments totally
bogus.
Noah Misch
Diffstat (limited to 'src/backend/utils/time/tqual.c')
-rw-r--r-- | src/backend/utils/time/tqual.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/backend/utils/time/tqual.c b/src/backend/utils/time/tqual.c index 727e0bf91bf..01f73980aff 100644 --- a/src/backend/utils/time/tqual.c +++ b/src/backend/utils/time/tqual.c @@ -10,12 +10,12 @@ * the passed-in buffer. The caller must hold not only a pin, but at least * shared buffer content lock on the buffer containing the tuple. * - * NOTE: must check TransactionIdIsInProgress (which looks in PGPROC array) + * NOTE: must check TransactionIdIsInProgress (which looks in PGXACT array) * before TransactionIdDidCommit/TransactionIdDidAbort (which look in * pg_clog). Otherwise we have a race condition: we might decide that a * just-committed transaction crashed, because none of the tests succeed. * xact.c is careful to record commit/abort in pg_clog before it unsets - * MyProc->xid in PGPROC array. That fixes that problem, but it also + * MyPgXact->xid in PGXACT array. That fixes that problem, but it also * means there is a window where TransactionIdIsInProgress and * TransactionIdDidCommit will both return true. If we check only * TransactionIdDidCommit, we could consider a tuple committed when a |