Re: [PATCH] Btree BackwardScan race condition on Standby during VACUUM - Mailing list pgsql-hackers

From Peter Geoghegan
Subject Re: [PATCH] Btree BackwardScan race condition on Standby during VACUUM
Date
Msg-id CAH2-Wzm7T_O+VUeo7=_NGPncs08z3JEybEwVLZGaASnbfg5vDA@mail.gmail.com
Whole thread Raw
In response to Re: [PATCH] Btree BackwardScan race condition on Standby duringVACUUM  ("Andrey M. Borodin" <x4mmm@yandex-team.ru>)
Responses Re: [PATCH] Btree BackwardScan race condition on Standby duringVACUUM  ("Andrey M. Borodin" <x4mmm@yandex-team.ru>)
List pgsql-hackers
On Mon, Mar 16, 2020 at 10:20 PM Andrey M. Borodin <x4mmm@yandex-team.ru> wrote:
> It seems to me that it's exactly the same check that I was trying to verify in amcheck patch [0].
> But there it was verified inside amcheck, but here it is verified by index scan.

Maybe we can accept your patch after fixing this bug. My objection to
the patch was that it couples locks in a way that's not compatible
with btree_xlog_unlink_page(). But the problem now seems to have been
btree_xlog_unlink_page() itself. It's possible that there are problems
elsewhere, but my recollection is that btree_xlog_unlink_page() was
the problem.

-- 
Peter Geoghegan



pgsql-hackers by date:

Previous
From: Peter Geoghegan
Date:
Subject: Re: [PATCH] Btree BackwardScan race condition on Standby during VACUUM
Next
From: Mark Dilger
Date:
Subject: Re: Adding missing object access hook invocations