Locate clearing and populating _periodicTaskHandles list in same method for readability.

1 files changed, 1 insertions, 2 deletions

diff --git a/searchcore/src/vespa/searchcore/proton/server/maintenancecontroller.cpp b/searchcore/src/vespa/searchcore/proton/server/maintenancecontroller.cpp
index 8f13dac8f6f..c86019005dc 100644
--- a/searchcore/src/vespa/searchcore/proton/server/maintenancecontroller.cpp
+++ b/searchcore/src/vespa/searchcore/proton/server/maintenancecontroller.cpp
@@ -143,14 +143,13 @@ MaintenanceController::restart()
     if (!getStarted() || getStopping() || !_readySubDB.valid()) {
         return;
     }
-    _periodicTaskHandles.clear();
-
     addJobsToPeriodicTimer();
 }
 
 void
 MaintenanceController::addJobsToPeriodicTimer()
 {
+    _periodicTaskHandles.clear();
     // No need to take _jobsLock as modification of _jobs also happens in master write thread.
     for (const auto &jw : _jobs) {
         const IMaintenanceJob &job = jw->getJob();