At Thu, 25 Jun 2026 15:19:57 +0900, Michael Paquier <[email protected]> wrote 
in 
> FWIW, even after sleeping on it, I don't think that this patch is
> going in the right direction.  I am pretty sure that we should just
> lift the ACL check in get_all_vacuum_rels() and always require
> vacuum_is_permitted_for_relation() to have the relation locked when
> called.  This way we would rely on one single code path for the ACL
> check, even if it means holding a reference of a relation OID for
> something to be processsed later.  So I would revisit the choice made

FWIW, I agree with this direction.

Regards,

-- 
Kyotaro Horiguchi
NTT Open Source Software Center


Reply via email to