Re: parallel vacuum - few questions on docs, comments and code
Bharath Rupireddy <bharath.rupireddyforpostgres@gmail.com>
From: Bharath Rupireddy <bharath.rupireddyforpostgres@gmail.com>
To: Masahiko Sawada <sawada.mshk@gmail.com>
Cc: Amit Kapila <amit.kapila16@gmail.com>, PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>
Date: 2021-05-13T15:30:26Z
Lists: pgsql-hackers
Attachments
- v1-0001-Parallel-Vacuum-Reword-Error-Messages-and-Docs.patch (application/x-patch) patch v1-0001
On Thu, May 13, 2021 at 7:00 PM Masahiko Sawada <sawada.mshk@gmail.com> wrote: > > > Yeah, I get it. Even if users don't specify a parallel option there > > > are chances that parallelism is picked. So the parallel degree is the > > > final number of workers that are chosen by the server for vacuuming > > > indexes. And, I think that parallel degree is something internal to > > > the server, and it's better we replace it in the vacuumdb.sgml, change > > > PARALLEL_DEGREE to PARALLEL_WORKERS in vacuumdb.c and change the error > > > message "parallel vacuum degree must be a non-negative integer" to > > > "parallel workers for vacuum must be greater than or equal to zero". > > > > > > Thoughts? > > I'm fine with this change. Thanks. > is important here but another idea to improve the error message would > be to change "parallel vacuum degree must be between 0 and %d” to "the > number of parallel workers must be between 0 and %d” (or using > “parallel workers for vacuum” instead of “the number of parallel > workers”) while leaving another message as it is. Done that way. PSA patch. With Regards, Bharath Rupireddy. EnterpriseDB: http://www.enterprisedb.com
Commits
-
Improve docs and error messages for parallel vacuum.
- 9012e5594c6f 13.4 landed
- 0734b0e98344 14.0 landed