On Tue, Mar 19, 2019 at 7:26 AM Masahiko Sawada <sawada.m...@gmail.com> wrote: > In parsing vacuum command, since only PARALLEL option can have an > argument I've added the check in ExecVacuum to erroring out when other > options have an argument. But it might be good to make other vacuum > options (perhaps except for DISABLE_PAGE_SKIPPING option) accept an > argument just like EXPLAIN command.
I think all of the existing options, including DISABLE_PAGE_SKIPPING, should permit an argument that is passed to defGetBoolean(). -- Robert Haas EnterpriseDB: http://www.enterprisedb.com The Enterprise PostgreSQL Company