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