On Tue, Mar 31, 2015 at 1:28 AM, Kyotaro HORIGUCHI <
horiguchi.kyot...@lab.ntt.co.jp> wrote:

> Hi, this is a bug in the commit 0d831389749a3baaced7b984205b9894a82444b9 .
>
> It allows vucuum freeze to be skipped and inversely lets regular
> vacuum wait for lock. The attched patch fixes it.
>
>
> In table_recheck_autovac, vacuum options are determined as following,
>
>  >      tab->at_vacoptions = VACOPT_SKIPTOAST |
>  >              (dovacuum ? VACOPT_VACUUM : 0) |
>  >              (doanalyze ? VACOPT_ANALYZE : 0) |
> !>              (wraparound ? VACOPT_NOWAIT : 0);
>
> The line prefixed by '!' looks inverted.
>

Thanks, it is obvious once you see it!

Your patch solved the problem for me.

Cheers,

Jeff

Reply via email to