From 2a8052a1aab823e2b7eb076700db7d8876220253 Mon Sep 17 00:00:00 2001
From: Wiebe Cazemier <wiebe@halfgaar.net>
Date: Wed, 13 Dec 2023 21:57:10 +0100
Subject: [PATCH] Remove config option
 rebuild_subscription_tree_interval_seconds

This was temporarily implemented as work-around. There is now a better
implementation.
---
 configfileparser.cpp | 15 ---------------
 settings.h           |  1 -
 2 files changed, 16 deletions(-)

diff --git a/configfileparser.cpp b/configfileparser.cpp
index f1af756a..d9ab06f2 100644
--- a/configfileparser.cpp
+++ b/configfileparser.cpp
@@ -898,21 +898,6 @@ void ConfigFileParser::loadFile(bool test)
 
                     tmpSettings.retainedMessagesDeliveryLimit = newVal;
                 }
-
-                if (testKeyValidity(key, "rebuild_subscription_tree_interval_seconds", validKeys))
-                {
-                    const uint32_t minVal = 900;
-                    const uint32_t newVal = std::stoul(value);
-
-                    if (newVal < minVal)
-                    {
-                        std::ostringstream oss;
-                        oss << "Value '" << value << "' for '" << key << "' is too low. It must be at least " << minVal;
-                        throw ConfigFileException(oss.str());
-                    }
-
-                    tmpSettings.rebuildSubscriptionTreeInterval = std::chrono::seconds(newVal);
-                }
             }
         }
         catch (std::invalid_argument &ex) // catch for the stoi()
diff --git a/settings.h b/settings.h
index aac3abae..f2b3cc41 100644
--- a/settings.h
+++ b/settings.h
@@ -80,7 +80,6 @@ class Settings
     uint maxQosBytesPendingPerClient = 65536;
     bool willsEnabled = true;
     int retainedMessagesDeliveryLimit = 2048;
-    std::chrono::seconds rebuildSubscriptionTreeInterval = std::chrono::minutes(30);
     RetainedMessagesMode retainedMessagesMode = RetainedMessagesMode::Enabled;
     SharedSubscriptionTargeting sharedSubscriptionTargeting = SharedSubscriptionTargeting::RoundRobin;
     std::list<std::shared_ptr<Listener>> listeners; // Default one is created later, when none are defined.