Hello, At Wed, 1 Jul 2026 08:35:12 +0900, Michael Paquier <[email protected]> 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
