Marginal cleanup in arrangements for ensuring StrategyHintVacuum is cleared
authorTom Lane <tgl@sss.pgh.pa.us>
Sun, 17 Sep 2006 22:16:22 +0000 (22:16 +0000)
committerTom Lane <tgl@sss.pgh.pa.us>
Sun, 17 Sep 2006 22:16:22 +0000 (22:16 +0000)
commit20f0f103cc2259f2cf8df15d0066c43abe71b64b
tree2014bc7d86927d53ceeaf43b27c39f3c99ac4fb3
parent895e7f0be8f2dc17b62fc45218f6f57b844d955f
Marginal cleanup in arrangements for ensuring StrategyHintVacuum is cleared
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.
src/backend/commands/vacuum.c
src/backend/storage/buffer/bufmgr.c