From 316d7599c0b22d59817a316263387a7cd5586466 Mon Sep 17 00:00:00 2001 From: naisila Date: Sat, 6 Aug 2022 23:42:13 +0300 Subject: [PATCH] Adds citus.mitmfifo GUC Previously we setting this configuration parameter in the fly for failure tests schedule. However, PG15 doesn't allow that anymore: reserved prefixes like "citus" cannot be used to set non-existing GUCs. Relevant PG commit: 88103567cb8fa5be46dc9fac3e3b8774951a2be7 --- src/backend/distributed/shared_library_init.c | 19 +++++++++++++++++++ 1 file changed, 19 insertions(+) diff --git a/src/backend/distributed/shared_library_init.c b/src/backend/distributed/shared_library_init.c index dd79f8d1e..a7315bcff 100644 --- a/src/backend/distributed/shared_library_init.c +++ b/src/backend/distributed/shared_library_init.c @@ -146,6 +146,7 @@ DEFINE_COLUMNAR_PASSTHROUGH_FUNC(test_columnar_storage_write_new_page) #define DUMMY_REAL_TIME_EXECUTOR_ENUM_VALUE 9999999 static char *CitusVersion = CITUS_VERSION; static char *DeprecatedEmptyString = ""; +static char *MitmfifoEmptyString = ""; /* deprecated GUC value that should not be used anywhere outside this file */ static int ReplicationModel = REPLICATION_MODEL_STREAMING; @@ -1767,6 +1768,24 @@ RegisterCitusConfigVariables(void) GUC_UNIT_MS | GUC_NO_SHOW_ALL, NULL, NULL, NULL); + /* + * Previously we setting this configuration parameter + * in the fly for failure tests schedule. + * However, PG15 doesn't allow that anymore: reserved prefixes + * like "citus" cannot be used to set non-existing GUCs. + * Relevant PG commit: 88103567cb8fa5be46dc9fac3e3b8774951a2be7 + */ + + DefineCustomStringVariable( + "citus.mitmfifo", + gettext_noop("Sets the citus mitm fifo path"), + NULL, + &MitmfifoEmptyString, + "", + PGC_SUSET, + GUC_NO_SHOW_ALL, + NULL, NULL, NULL); + DefineCustomEnumVariable( "citus.multi_shard_modify_mode", gettext_noop("Sets the connection type for multi shard modify queries"),