diff options
author | jonmv <venstad@gmail.com> | 2023-08-31 08:25:32 +0200 |
---|---|---|
committer | jonmv <venstad@gmail.com> | 2023-08-31 08:25:32 +0200 |
commit | eb479a5207ea2b536b8d4cc78afa17203ff29be7 (patch) | |
tree | db36e18af7b486d05e48a5f010fe0bac2fce0d3f /container-search/src/main/java/com/yahoo/search/logging/AbstractSpoolingLogger.java | |
parent | 2d3a1be956b24f3eda343bddcecea6b418f4cd7c (diff) |
Shut down loggers on deconstruct, and do this in unit test instead of sleep
Diffstat (limited to 'container-search/src/main/java/com/yahoo/search/logging/AbstractSpoolingLogger.java')
-rw-r--r-- | container-search/src/main/java/com/yahoo/search/logging/AbstractSpoolingLogger.java | 5 |
1 files changed, 3 insertions, 2 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 601da11ab33..0fcc54493e3 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 @@ -54,8 +54,9 @@ public abstract class AbstractSpoolingLogger extends AbstractThreadedLogger impl return true; } - // TODO Call from a component or make this class a component - public void shutdown() { + @Override + public void deconstruct() { + super.deconstruct(); executorService.shutdown(); try { if ( ! executorService.awaitTermination(10, TimeUnit.SECONDS)) |