postgrespro.git / summary / log / commit / refs

commit    7c366ac969ce060c4080fbdfe74a0131ae4bf6bb
Author:   Tom Lane <tgl@sss.pgh.pa.us>
Date:     Tue Mar 26 17:32:30 2019 +0000

    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 3b0b138f24..b58a1f7a72 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",


[parent: 1d21ba8a9b8c]