Re: collect_corrupt_items_vacuum.patch

Hi!

I agree with Alexander Lakhin about PROC_IN_VACUUM and
VISHORIZON_DATA_STRICT:
1) probably manipulations with the PROC_IN_VACUUM flag in
pg_visibility.c were needed for condition [1] and can be removed now;
2) the VISHORIZON_DATA_STRICT macro is probably unnecessary too (since
we are not going to use it in the GlobalVisHorizonKindForRel() function).

Also It would be nice to remove the get_strict_xid_horizon() function
from the comment (replace to GetStrictOldestNonRemovableTransactionId()?).

[1]
https://github.com/postgres/postgres/blob/4d0cf0b05defcee985d5af38cb0db2b9c2f8dbae/src/backend/storage/ipc/procarray.c#L1812

--
With best regards,
Dmitry Koval

Postgres Professional: http://postgrespro.com