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/snapmgr.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/snapmgr.c')
| -rw-r--r-- | src/backend/utils/time/snapmgr.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/src/backend/utils/time/snapmgr.c b/src/backend/utils/time/snapmgr.c index 5aebbd18552..574099dc9a3 100644 --- a/src/backend/utils/time/snapmgr.c +++ b/src/backend/utils/time/snapmgr.c @@ -19,7 +19,7 @@ * have regd_count = 1 and are counted in RegisteredSnapshots, but are not * tracked by any resource owner. * - * These arrangements let us reset MyProc->xmin when there are no snapshots + * These arrangements let us reset MyPgXact->xmin when there are no snapshots * referenced by this transaction. (One possible improvement would be to be * able to advance Xmin when the snapshot with the earliest Xmin is no longer * referenced. That's a bit harder though, it requires more locking, and @@ -104,7 +104,7 @@ static ActiveSnapshotElt *ActiveSnapshot = NULL; * How many snapshots is resowner.c tracking for us? * * Note: for now, a simple counter is enough. However, if we ever want to be - * smarter about advancing our MyProc->xmin we will need to be more + * smarter about advancing our MyPgXact->xmin we will need to be more * sophisticated about this, perhaps keeping our own list of snapshots. */ static int RegisteredSnapshots = 0; @@ -266,7 +266,7 @@ SetTransactionSnapshot(Snapshot sourcesnap, TransactionId sourcexid) /* NB: curcid should NOT be copied, it's a local matter */ /* - * Now we have to fix what GetSnapshotData did with MyProc->xmin and + * Now we have to fix what GetSnapshotData did with MyPgXact->xmin and * TransactionXmin. There is a race condition: to make sure we are not * causing the global xmin to go backwards, we have to test that the * source transaction is still running, and that has to be done atomically. @@ -569,7 +569,7 @@ UnregisterSnapshotFromOwner(Snapshot snapshot, ResourceOwner owner) /* * SnapshotResetXmin * - * If there are no more snapshots, we can reset our PGPROC->xmin to InvalidXid. + * If there are no more snapshots, we can reset our PGXACT->xmin to InvalidXid. * Note we can do this without locking because we assume that storing an Xid * is atomic. */ |