From 7c366ac969ce060c4080fbdfe74a0131ae4bf6bb Mon Sep 17 00:00:00 2001 From: Tom Lane Date: Tue, 26 Mar 2019 13:32:30 -0400 Subject: [PATCH] Fix oversight in data-type change for autovacuum_vacuum_cost_delay. Commit caf626b2c missed that the relevant reloptions entry needs to be moved from the intRelOpts[] array to realRelOpts[]. Somewhat surprisingly, it seems to work anyway, perhaps because the desired default and limit values are all integers. We ought to have either a simpler data structure or better cross-checking here, but that's for another patch. Nikolay Shaplov Discussion: https://postgr.es/m/4861742.12LTaSB3sv@x200m --- src/backend/access/common/reloptions.c | 18 +++++++++--------- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/src/backend/access/common/reloptions.c b/src/backend/access/common/reloptions.c index 3b0b138f247..b58a1f7a729 100644 --- a/src/backend/access/common/reloptions.c +++ b/src/backend/access/common/reloptions.c @@ -212,15 +212,6 @@ static relopt_int intRelOpts[] = }, -1, 0, INT_MAX }, - { - { - "autovacuum_vacuum_cost_delay", - "Vacuum cost delay in milliseconds, for autovacuum", - RELOPT_KIND_HEAP | RELOPT_KIND_TOAST, - ShareUpdateExclusiveLock - }, - -1, 0, 100 - }, { { "autovacuum_vacuum_cost_limit", @@ -346,6 +337,15 @@ static relopt_int intRelOpts[] = static relopt_real realRelOpts[] = { + { + { + "autovacuum_vacuum_cost_delay", + "Vacuum cost delay in milliseconds, for autovacuum", + RELOPT_KIND_HEAP | RELOPT_KIND_TOAST, + ShareUpdateExclusiveLock + }, + -1, 0.0, 100.0 + }, { { "autovacuum_vacuum_scale_factor",