From 6599c9ac3340b6cd3d86a0a7f866b80a009fecab Mon Sep 17 00:00:00 2001 From: Robert Haas Date: Tue, 11 Apr 2017 13:03:44 -0400 Subject: Add an Assert() to max_parallel_workers enforcement. To prevent future bugs along the lines of the one corrected by commit 8ff518699f19dd0a5076f5090bac8400b8233f7f, or find any that remain in the current code, add an Assert() that the difference between parallel_register_count and parallel_terminate_count is in a sane range. Kuntal Ghosh, with considerable tidying-up by me, per a suggestion from Neha Khatri. Reviewed by Tomas Vondra. Discussion: http://postgr.es/m/CAFO0U+-E8yzchwVnvn5BeRDPgX2z9vZUxQ8dxx9c0XFGBC7N1Q@mail.gmail.com --- src/backend/utils/misc/guc.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) (limited to 'src/backend/utils/misc/guc.c') diff --git a/src/backend/utils/misc/guc.c b/src/backend/utils/misc/guc.c index 6e39a676094..9ad8361a9bd 100644 --- a/src/backend/utils/misc/guc.c +++ b/src/backend/utils/misc/guc.c @@ -57,7 +57,7 @@ #include "parser/scansup.h" #include "pgstat.h" #include "postmaster/autovacuum.h" -#include "postmaster/bgworker.h" +#include "postmaster/bgworker_internals.h" #include "postmaster/bgwriter.h" #include "postmaster/postmaster.h" #include "postmaster/syslogger.h" @@ -2713,7 +2713,7 @@ static struct config_int ConfigureNamesInt[] = NULL }, &max_parallel_workers_per_gather, - 2, 0, 1024, + 2, 0, MAX_PARALLEL_WORKER_LIMIT, NULL, NULL, NULL }, @@ -2723,7 +2723,7 @@ static struct config_int ConfigureNamesInt[] = NULL }, &max_parallel_workers, - 8, 0, 1024, + 8, 0, MAX_PARALLEL_WORKER_LIMIT, NULL, NULL, NULL }, -- cgit v1.2.3