diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2007-04-03 16:34:36 +0000 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2007-04-03 16:34:36 +0000 |
| commit | 9c9b619473cd9d2f3f3181bd9cd6862c64abb4d3 (patch) | |
| tree | 4cf41b7316d638094c0f30c8e2c0b6565d4465ee /src/include/storage/proc.h | |
| parent | fb4279e99cf71c27c132a75ed62a08395329b496 (diff) | |
| download | postgresql-9c9b619473cd9d2f3f3181bd9cd6862c64abb4d3.tar.gz postgresql-9c9b619473cd9d2f3f3181bd9cd6862c64abb4d3.zip | |
Remove the CheckpointStartLock in favor of having backends show whether they
are in their commit critical sections via flags in the ProcArray. Checkpoint
can watch the ProcArray to determine when it's safe to proceed. This is
a considerably better solution to the original problem of race conditions
between checkpoint and transaction commit: it speeds up commit, since there's
one less lock to fool with, and it prevents the problem of checkpoint being
delayed indefinitely when there's a constant flow of commits. Heikki, with
some kibitzing from Tom.
Diffstat (limited to 'src/include/storage/proc.h')
| -rw-r--r-- | src/include/storage/proc.h | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/src/include/storage/proc.h b/src/include/storage/proc.h index 2b20eda828d..772cf52cdf9 100644 --- a/src/include/storage/proc.h +++ b/src/include/storage/proc.h @@ -7,7 +7,7 @@ * Portions Copyright (c) 1996-2007, PostgreSQL Global Development Group * Portions Copyright (c) 1994, Regents of the University of California * - * $PostgreSQL: pgsql/src/include/storage/proc.h,v 1.96 2007/03/07 13:35:03 alvherre Exp $ + * $PostgreSQL: pgsql/src/include/storage/proc.h,v 1.97 2007/04/03 16:34:36 tgl Exp $ * *------------------------------------------------------------------------- */ @@ -74,6 +74,8 @@ struct PGPROC Oid databaseId; /* OID of database this backend is using */ Oid roleId; /* OID of role using this backend */ + bool inCommit; /* true if within commit critical section */ + bool inVacuum; /* true if current xact is a LAZY VACUUM */ bool isAutovacuum; /* true if it's autovacuum */ |