Merge pull request #24578 from vespa-engine/hmusum/executor-cannot-be-static

Executor cannot be static, will not work if container is reconfigured

1 files changed, 2 insertions, 3 deletions

diff --git a/container-search/src/main/java/com/yahoo/search/logging/AbstractSpoolingLogger.java b/container-search/src/main/java/com/yahoo/search/logging/AbstractSpoolingLogger.java
index 9f07d56fe28..7641b04addf 100644
--- a/container-search/src/main/java/com/yahoo/search/logging/AbstractSpoolingLogger.java
+++ b/container-search/src/main/java/com/yahoo/search/logging/AbstractSpoolingLogger.java
@@ -20,9 +20,7 @@ public abstract class AbstractSpoolingLogger extends AbstractThreadedLogger impl
 
     protected static final java.util.logging.Logger log = java.util.logging.Logger.getLogger(Spooler.class.getName());
 
-    private static final ScheduledExecutorService executorService =
-            new ScheduledThreadPoolExecutor(1, new DaemonThreadFactory("AbstractSpoolingLogger-send-"));
-
+    private final ScheduledExecutorService executorService;
     protected final Spooler spooler;
 
     public AbstractSpoolingLogger() {
@@ -31,6 +29,7 @@ public abstract class AbstractSpoolingLogger extends AbstractThreadedLogger impl
 
     public AbstractSpoolingLogger(Spooler spooler) {
         this.spooler = spooler;
+        this.executorService = new ScheduledThreadPoolExecutor(1, new DaemonThreadFactory("AbstractSpoolingLogger-send-"));
         executorService.scheduleWithFixedDelay(this, 0, 10L, TimeUnit.MILLISECONDS);
     }