vacuumdb: Fix comment for vacuum_one_database().

nathan-bossart · nathan-bossart · commit 6a9b2a631aa3 · 2025-01-17T15:23:14.000-06:00
Since commit e0c2933, vacuum_one_database() always uses a catalog query to discover the tables to process, but this comment still notes the special case for which we used a catalog query before that commit. Let's just remove that note. Also, commit 7781f4e renamed the "tables" parameter to "objects" but missed updating this comment. This commit fixes that as well.
diff --git a/src/bin/scripts/vacuumdb.c b/src/bin/scripts/vacuumdb.c
@@ -461,15 +461,14 @@ escape_quotes(const char *src)
 /*
  * vacuum_one_database
  *
- * Process tables in the given database.  If the 'tables' list is empty,
+ * Process tables in the given database.  If the 'objects' list is empty,
  * process all tables in the database.
  *
  * Note that this function is only concerned with running exactly one stage
  * when in analyze-in-stages mode; caller must iterate on us if necessary.
  *
  * If concurrentCons is > 1, multiple connections are used to vacuum tables
- * in parallel.  In this case and if the table list is empty, we first obtain
- * a list of tables from the database.
+ * in parallel.
  */
 static void
 vacuum_one_database(ConnParams *cparams,

Original file line number	Diff line number	Diff line change
`@@ -461,15 +461,14 @@ escape_quotes(const char *src)`
`461`	`461`	`/*`
`462`	`462`	`* vacuum_one_database`
`463`	`463`	`*`
`464`		`- * Process tables in the given database. If the 'tables' list is empty,`
	`464`	`+ * Process tables in the given database. If the 'objects' list is empty,`
`465`	`465`	`* process all tables in the database.`
`466`	`466`	`*`
`467`	`467`	`* Note that this function is only concerned with running exactly one stage`
`468`	`468`	`* when in analyze-in-stages mode; caller must iterate on us if necessary.`
`469`	`469`	`*`
`470`	`470`	`* If concurrentCons is > 1, multiple connections are used to vacuum tables`
`471`		`- * in parallel. In this case and if the table list is empty, we first obtain`
`472`		`- * a list of tables from the database.`
	`471`	`+ * in parallel.`
`473`	`472`	`*/`
`474`	`473`	`static void`
`475`	`474`	`vacuum_one_database(ConnParams *cparams,`