From 62c5090fd85582892a105fa4cb0669c510ec7d19 Mon Sep 17 00:00:00 2001
From: ivyazmitinov <ivanvyazmitinov@gmail.com>
Date: Wed, 7 Feb 2024 09:20:43 +0100
Subject: [PATCH] Fix gucs order

---
 src/backend/distributed/shared_library_init.c | 52 +++++++++----------
 1 file changed, 26 insertions(+), 26 deletions(-)

diff --git a/src/backend/distributed/shared_library_init.c b/src/backend/distributed/shared_library_init.c
index f06acf009..c672c26a3 100644
--- a/src/backend/distributed/shared_library_init.c
+++ b/src/backend/distributed/shared_library_init.c
@@ -1933,6 +1933,20 @@ RegisterCitusConfigVariables(void)
 		GUC_STANDARD,
 		NULL, NULL, NULL);
 
+	DefineCustomIntVariable(
+			"citus.max_databases_per_worker_tracked",
+			gettext_noop("Sets the amount of databases per worker tracked."),
+			gettext_noop(
+					"This configuration value complements the citus.max_worker_nodes_tracked."
+					"It should be used when there are more then one database with Citus in cluster,"
+					"and, effectively, limits the size of the hash table with connections per worker + database."
+					"Currently, it does not affect the connection management logic and serves only statistical purposes."),
+			&MaxDatabasesPerWorkerNodesTracked,
+			1, 1, INT_MAX,
+			PGC_POSTMASTER,
+			GUC_STANDARD,
+			NULL, NULL, NULL);
+
 	DefineCustomIntVariable(
 		"citus.max_high_priority_background_processes",
 		gettext_noop("Sets the maximum number of background processes "
@@ -1958,6 +1972,18 @@ RegisterCitusConfigVariables(void)
 		GUC_UNIT_KB | GUC_STANDARD,
 		NULL, NULL, NULL);
 
+	DefineCustomIntVariable(
+			"citus.max_maintenance_shared_pool_size",
+			gettext_noop("Similar to citus.max_shared_pool_size, but applies to connections "
+						 "for maintenance operations only."
+						 "Setting it to 0 or -1 disables maintenance connection throttling."),
+			NULL,
+			&MaxMaintenanceSharedPoolSize,
+			5, -1, INT_MAX,
+			PGC_SIGHUP,
+			GUC_SUPERUSER_ONLY,
+			NULL, NULL, NULL);
+
 	DefineCustomIntVariable(
 		"citus.max_matview_size_to_auto_recreate",
 		gettext_noop("Sets the maximum size of materialized views in MB to "
@@ -1993,18 +2019,6 @@ RegisterCitusConfigVariables(void)
 		GUC_SUPERUSER_ONLY,
 		NULL, NULL, MaxSharedPoolSizeGucShowHook);
 
-	DefineCustomIntVariable(
-		"citus.max_maintenance_shared_pool_size",
-		gettext_noop("Similar to citus.max_shared_pool_size, but applies to connections "
-					 "for maintenance operations only."
-					 "Setting it to 0 or -1 disables maintenance connection throttling."),
-		NULL,
-		&MaxMaintenanceSharedPoolSize,
-		5, -1, INT_MAX,
-		PGC_SIGHUP,
-		GUC_SUPERUSER_ONLY,
-		NULL, NULL, NULL);
-
 	DefineCustomIntVariable(
 		"citus.max_worker_nodes_tracked",
 		gettext_noop("Sets the maximum number of worker nodes that are tracked."),
@@ -2024,20 +2038,6 @@ RegisterCitusConfigVariables(void)
 		GUC_STANDARD,
 		NULL, NULL, NULL);
 
-	DefineCustomIntVariable(
-		"citus.max_databases_per_worker_tracked",
-		gettext_noop("Sets the amount of databases per worker tracked."),
-		gettext_noop(
-			"This configuration value complements the citus.max_worker_nodes_tracked."
-			"It should be used when there are more then one database with Citus in cluster,"
-			"and, effectively, limits the size of the hash table with connections per worker + database."
-			"Currently, it does not affect the connection management logic and serves only statistical purposes."),
-		&MaxDatabasesPerWorkerNodesTracked,
-		1, 1, INT_MAX,
-		PGC_POSTMASTER,
-		GUC_STANDARD,
-		NULL, NULL, NULL);
-
 	DefineCustomIntVariable(
 		"citus.metadata_sync_interval",
 		gettext_noop("Sets the time to wait between metadata syncs."),