On Wed, Jun 14, 2023 at 08:16:15AM +0900, Michael Paquier wrote: > While on it, this buzzes me: > static bool > -vacuum_rel(Oid relid, RangeVar *relation, VacuumParams *params) > +vacuum_rel(Oid relid, RangeVar *relation, VacuumParams *params, bool > skip_privs) > > VacuumParams has been originally introduced to avoid extending > vacuum_rel() with a bunch of arguments, no?
Yeah, that could probably be moved into VacuumParams. > So, yes, agreed about the removal of has_partition_ancestor_privs(). > I am adding an open item assigned to you and Jeff. Thanks. I suspect there's more discussion incoming, but I'm hoping to close this item one way or another by 16beta2. -- Nathan Bossart Amazon Web Services: https://aws.amazon.com
