Make sure we don't skip databases that are supposed to be vacuumed "exactly

alvherre · alvherre · commit 067deaf83d60 · 2007-05-07T20:41:24.000Z
now".  This can happen if the time granularity is not very high.

Per ITAGAKI Takahiro.
diff --git 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.45 2007/05/04 02:06:13 tgl Exp $
+ *	  $PostgreSQL: pgsql/src/backend/postmaster/autovacuum.c,v 1.46 2007/05/07 20:41:24 alvherre Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -1036,8 +1036,8 @@ do_start_worker(void)
 				 * Skip this database if its next_worker value falls between
 				 * the current time and the current time plus naptime.
 				 */
-				if (TimestampDifferenceExceeds(current_time,
-											   dbp->adl_next_worker, 0) &&
+				if (!TimestampDifferenceExceeds(dbp->adl_next_worker,
+											   current_time, 0) &&
 					!TimestampDifferenceExceeds(current_time,
 												dbp->adl_next_worker,
 												autovacuum_naptime * 1000))

Original file line number	Diff line number	Diff line change
`@@ -10,7 +10,7 @@`
`10`	`10`	`*`
`11`	`11`	`*`
`12`	`12`	`* IDENTIFICATION`
`13`		`- * $PostgreSQL: pgsql/src/backend/postmaster/autovacuum.c,v 1.45 2007/05/04 02:06:13 tgl Exp $`
	`13`	`+ * $PostgreSQL: pgsql/src/backend/postmaster/autovacuum.c,v 1.46 2007/05/07 20:41:24 alvherre Exp $`
`14`	`14`	`*`
`15`	`15`	`*-------------------------------------------------------------------------`
`16`	`16`	`*/`
`@@ -1036,8 +1036,8 @@ do_start_worker(void)`
`1036`	`1036`	`* Skip this database if its next_worker value falls between`
`1037`	`1037`	`* the current time and the current time plus naptime.`
`1038`	`1038`	`*/`
`1039`		`- if (TimestampDifferenceExceeds(current_time,`
`1040`		`- dbp->adl_next_worker, 0) &&`
	`1039`	`+ if (!TimestampDifferenceExceeds(dbp->adl_next_worker,`
	`1040`	`+ current_time, 0) &&`
`1041`	`1041`	`!TimestampDifferenceExceeds(current_time,`
`1042`	`1042`	`dbp->adl_next_worker,`
`1043`	`1043`	`autovacuum_naptime * 1000))`