Hi, While reading autovacuum.c, I found the following comments in AutoVacLauncherMain() right before the call to rebuild_database_list();
* Create the initial database list. The invariant we want this list to * keep is that it's ordered by decreasing next_time. As soon as an entry However, I couldn't fnd any variable named "next_time". I believe it actually refers to adl_next_worker and the comments should use "next_worker" as is done in other nereby comments. I've attahed a patch to address this. Regards, Yugo Nagata -- Yugo Nagata <nag...@sraoss.co.jp>
diff --git a/src/backend/postmaster/autovacuum.c b/src/backend/postmaster/autovacuum.c index 9474095f271..5161ec39d72 100644 --- a/src/backend/postmaster/autovacuum.c +++ b/src/backend/postmaster/autovacuum.c @@ -562,7 +562,7 @@ AutoVacLauncherMain(const void *startup_data, size_t startup_data_len) /* * Create the initial database list. The invariant we want this list to - * keep is that it's ordered by decreasing next_time. As soon as an entry + * keep is that it's ordered by decreasing next_worker. As soon as an entry * is updated to a higher time, it will be moved to the front (which is * correct because the only operation is to add autovacuum_naptime to the * entry, and time always increases).