after an error during VACUUM. We have a PG_TRY block anyway around the only
call sites, so just reset it in the CATCH clause instead of having
AtEOXact_Buffers blindly do it during xact end. I think the old code was
actively wrong for the case of a failure during ANALYZE inside a
subtransaction --- the flag wouldn't get cleared until main transaction end.
Probably not worth back-patching though.
*
*
* IDENTIFICATION
- * $PostgreSQL: pgsql/src/backend/commands/vacuum.c,v 1.338 2006/08/18 16:09:08 tgl Exp $
+ * $PostgreSQL: pgsql/src/backend/commands/vacuum.c,v 1.339 2006/09/17 22:16:22 tgl Exp $
*
*-------------------------------------------------------------------------
*/
{
/* Make sure cost accounting is turned off after error */
VacuumCostActive = false;
+ /* And reset buffer replacement strategy, too */
+ StrategyHintVacuum(false);
PG_RE_THROW();
}
PG_END_TRY();
* Now release the session-level lock on the master table.
*/
UnlockRelationIdForSession(&onerelid, lmode);
-
- return;
}
*
*
* IDENTIFICATION
- * $PostgreSQL: pgsql/src/backend/storage/buffer/bufmgr.c,v 1.209 2006/07/23 03:07:58 tgl Exp $
+ * $PostgreSQL: pgsql/src/backend/storage/buffer/bufmgr.c,v 1.210 2006/09/17 22:16:22 tgl Exp $
*
*-------------------------------------------------------------------------
*/
#endif
AtEOXact_LocalBuffers(isCommit);
-
- /* Make sure we reset the strategy hint in case VACUUM errored out */
- StrategyHintVacuum(false);
}
/*