Alvaro Herrera <alvherre@2ndquadrant.com> writes:
> But autovacuum is still manufacturing a VacuumStmt by hand. If we want
> to get rid of that, I think it'd work to have a new
> ExecVacuum(VacuumStmt, params) function which is called from
> standard_ProcessUtility and does just vacuum(rel, relid, params).
> Autovacuum on the other hand can call vacuum() without having to
> construct the parse node.
Why would we want to get rid of that? A struct is a handy and legible
way to pass a pile of parameters. I doubt it would be an improvement for
vacuum() to grow a long list of separate parameters.
regards, tom lane