From 0bd007c4f96597295c5f086138a7c381301ceeab Mon Sep 17 00:00:00 2001 From: Alvaro Herrera Date: Wed, 19 Mar 2008 14:18:21 +0000 Subject: [PATCH] We no longer need a snapshot set after opening the finishing transaction: this is redundant because autovacuum now always analyzes a single table per transaction. --- src/backend/commands/vacuum.c | 9 --------- 1 file changed, 9 deletions(-) diff --git a/src/backend/commands/vacuum.c b/src/backend/commands/vacuum.c index abd09a6499..3e9b2a2286 100644 --- a/src/backend/commands/vacuum.c +++ b/src/backend/commands/vacuum.c @@ -482,15 +482,6 @@ vacuum(VacuumStmt *vacstmt, List *relids, * PostgresMain(). */ StartTransactionCommand(); - - /* - * Re-establish the transaction snapshot. This is wasted effort when - * we are called as a normal utility command, because the new - * transaction will be dropped immediately by PostgresMain(); but it's - * necessary if we are called from autovacuum because autovacuum might - * continue on to do an ANALYZE-only call. - */ - ActiveSnapshot = CopySnapshot(GetTransactionSnapshot()); } if (vacstmt->vacuum && !IsAutoVacuumWorkerProcess()) -- 2.39.5