diff options
| author | Robert Haas <rhaas@postgresql.org> | 2011-11-25 08:02:10 -0500 |
|---|---|---|
| committer | Robert Haas <rhaas@postgresql.org> | 2011-11-25 08:02:10 -0500 |
| commit | ed0b409d22346b1b027a4c2099ca66984d94b6dd (patch) | |
| tree | 3d9942b4131ffa73cbf83735c3965b4c1fc7e307 /src/backend/commands/analyze.c | |
| parent | 9ed439a9c07b69c2617cc98596611fdbdc22472c (diff) | |
| download | postgresql-ed0b409d22346b1b027a4c2099ca66984d94b6dd.tar.gz postgresql-ed0b409d22346b1b027a4c2099ca66984d94b6dd.zip | |
Move "hot" members of PGPROC into a separate PGXACT array.
This speeds up snapshot-taking and reduces ProcArrayLock contention.
Also, the PGPROC (and PGXACT) structures used by two-phase commit are
now allocated as part of the main array, rather than in a separate
array, and we keep ProcArray sorted in pointer order. These changes
are intended to minimize the number of cache lines that must be pulled
in to take a snapshot, and testing shows a substantial increase in
performance on both read and write workloads at high concurrencies.
Pavan Deolasee, Heikki Linnakangas, Robert Haas
Diffstat (limited to 'src/backend/commands/analyze.c')
| -rw-r--r-- | src/backend/commands/analyze.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/backend/commands/analyze.c b/src/backend/commands/analyze.c index 32985a4a0a0..314324618a8 100644 --- a/src/backend/commands/analyze.c +++ b/src/backend/commands/analyze.c @@ -223,7 +223,7 @@ analyze_rel(Oid relid, VacuumStmt *vacstmt, BufferAccessStrategy bstrategy) * OK, let's do it. First let other backends know I'm in ANALYZE. */ LWLockAcquire(ProcArrayLock, LW_EXCLUSIVE); - MyProc->vacuumFlags |= PROC_IN_ANALYZE; + MyPgXact->vacuumFlags |= PROC_IN_ANALYZE; LWLockRelease(ProcArrayLock); /* @@ -250,7 +250,7 @@ analyze_rel(Oid relid, VacuumStmt *vacstmt, BufferAccessStrategy bstrategy) * because the vacuum flag is cleared by the end-of-xact code. */ LWLockAcquire(ProcArrayLock, LW_EXCLUSIVE); - MyProc->vacuumFlags &= ~PROC_IN_ANALYZE; + MyPgXact->vacuumFlags &= ~PROC_IN_ANALYZE; LWLockRelease(ProcArrayLock); } |