On Wed, 16 Sept 2026 at 15:23, Aleksander Alekseev
<[email protected]> wrote:
>
> Hi,
>
> The proposed patch removes dead code in src/backend/storage/lmgr/predicate.c
>
> If my understanding is correct, PageIsPredicateLocked() was added in
> dafaa3efb75 (Feb 2011) but its call was removed in 47ad79122bc (Mar
> 2011) and we have never used it since. It was mentioned in one recent
> discussion [1] but it was pointed out that it can't be used there
> either.
>
> If anyone believes that we should keep it alternatively we could cover
> it with tests or at least add an Assert() somewhere.

I'm fine with removing it, for reasons including because its signature
promise wasn't ever truly clear.

-Matthias


Reply via email to