Hello,
At Wed, 1 Jul 2026 08:35:12 +0900, Michael Paquier <michael@paquier.xyz> wrote in
> > 3. The Assert that Bharath suggested earlier doesn't apply as-is but can be
> > incorporated into the patch as:
> > Assert(missing_ok || CheckRelationOidLockedByMe(relid, AccessShareLock,
> > true));
>
> This fail-safe would be a welcome addition, yes. Perhaps Nathan feels
> differently, which would be also fine by me.
To me, this Assert seems worthwhile as part of the function's contract.
Regards,
--
Kyotaro Horiguchi
NTT Open Source Software Center