We no longer need to palloc the VacuumStmt node; keeping it on the stack is

simpler.

diff --git a/src/backend/postmaster/autovacuum.c b/src/backend/postmaster/autovacuum.c
index b29185583052d044efa0db2bd585139e6717100e..466f03a5f543389bbb6d289f2bb12f3a71e4a620 100644 (file)
--- a/src/backend/postmaster/autovacuum.c
+++ b/src/backend/postmaster/autovacuum.c
@@ -10,7 +10,7 @@
  *
  *
  * IDENTIFICATION
- *       $PostgreSQL: pgsql/src/backend/postmaster/autovacuum.c,v 1.34 2007/03/13 00:33:41 tgl Exp $
+ *       $PostgreSQL: pgsql/src/backend/postmaster/autovacuum.c,v 1.35 2007/03/23 20:56:39 alvherre Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -1237,32 +1237,28 @@ static void
 autovacuum_do_vac_analyze(Oid relid, bool dovacuum, bool doanalyze,
                                                  int freeze_min_age)
 {
-       VacuumStmt *vacstmt;
+       VacuumStmt      vacstmt;
        MemoryContext old_cxt;
 
        /*
-        * The node must survive transaction boundaries, so make sure we create it
+        * The list must survive transaction boundaries, so make sure we create it
         * in a long-lived context
         */
        old_cxt = MemoryContextSwitchTo(AutovacMemCxt);
 
-       vacstmt = makeNode(VacuumStmt);
-
        /* Set up command parameters */
-       vacstmt->vacuum = dovacuum;
-       vacstmt->full = false;
-       vacstmt->analyze = doanalyze;
-       vacstmt->freeze_min_age = freeze_min_age;
-       vacstmt->verbose = false;
-       vacstmt->relation = NULL;       /* not used since we pass a relids list */
-       vacstmt->va_cols = NIL;
+       vacstmt.vacuum = dovacuum;
+       vacstmt.full = false;
+       vacstmt.analyze = doanalyze;
+       vacstmt.freeze_min_age = freeze_min_age;
+       vacstmt.verbose = false;
+       vacstmt.relation = NULL;        /* not used since we pass a relids list */
+       vacstmt.va_cols = NIL;
 
        /* Let pgstat know what we're doing */
-       autovac_report_activity(vacstmt, relid);
-
-       vacuum(vacstmt, list_make1_oid(relid), true);
+       autovac_report_activity(&vacstmt, relid);
 
-       pfree(vacstmt);
+       vacuum(&vacstmt, list_make1_oid(relid), true);
        MemoryContextSwitchTo(old_cxt);
 }