]> granicus.if.org Git - postgresql/commit
Fix error handling of vacuumdb when running out of fds
authorMichael Paquier <michael@paquier.xyz>
Mon, 26 Aug 2019 02:14:44 +0000 (11:14 +0900)
committerMichael Paquier <michael@paquier.xyz>
Mon, 26 Aug 2019 02:14:44 +0000 (11:14 +0900)
commita21ec1a959ef2f7fcd14391c97a14f5f10630aff
treeec0f90618c4ce94c4cde367c4542dd2bdea1e01c
parent65b1cad5a0929e685e79f7e84f56e0f28a54107a
Fix error handling of vacuumdb when running out of fds

When trying to use a high number of jobs, vacuumdb has only checked for
a maximum number of jobs used, causing confusing failures when running
out of file descriptors when the jobs open connections to Postgres.
This commit changes the error handling so as we do not check anymore for
a maximum number of allowed jobs when parsing the option value with
FD_SETSIZE, but check instead if a file descriptor is within the
supported range when opening the connections for the jobs so as this is
detected at the earliest time possible.

Also, improve the error message to give a hint about the number of jobs
recommended, using a wording given by the reviewers of the patch.

Reported-by: Andres Freund
Author: Michael Paquier
Reviewed-by: Andres Freund, Álvaro Herrera, Tom Lane
Discussion: https://postgr.es/m/20190818001858.ho3ev4z57fqhs7a5@alap3.anarazel.de
Backpatch-through: 9.5
src/bin/scripts/vacuumdb.c