Re: VACUUM PARALLEL option vs. max_parallel_maintenance_workers
Peter Eisentraut <peter.eisentraut@2ndquadrant.com>
From: Peter Eisentraut <peter.eisentraut@2ndquadrant.com>
To: Amit Kapila <amit.kapila16@gmail.com>
Cc: pgsql-hackers <pgsql-hackers@postgresql.org>
Date: 2020-09-20T13:46:29Z
Lists: pgsql-hackers
On 2020-09-19 13:24, Amit Kapila wrote: >> I think the implemented behavior is wrong. > > It is the same as what we do for other parallel operations, for > example, we limit the number of parallel workers for parallel create > index by 'max_parallel_maintenance_workers' and parallel scan > operations are limited by 'max_parallel_workers_per_gather'. But in those cases we don't provide user-visible options to specify a per-command setting, so it's not the same thing, is it? >> The VACUUM PARALLEL option >> should override the max_parallel_maintenance_worker setting. >> >> Otherwise, what's the point of the command option? > > It is for the cases where the user has a better idea of workload. We > can launch only a limited number of parallel workers > 'max_parallel_workers' in the system, so sometimes users would like to > use it as per their requirement. Right, but my point is, it doesn't actually do that correctly. I can't just say, oh, I have a maintenance window, I'd like to run a really fast VACUUM. The PARALLEL option is capped by the setting you'd normally use anyway, so specifying it is useless. The only thing it can do right now is if you want to run a manual VACUUM less parallel than by default. But I don't see how that is often useful. -- Peter Eisentraut http://www.2ndQuadrant.com/ PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services
Commits
-
Change the docs for PARALLEL option of Vacuum.
- 99ae342fc4ff 13.1 landed
- 560d260d7852 14.0 landed