summaryrefslogtreecommitdiffstats
path: root/infra/impl
diff options
context:
space:
mode:
authorMarek Gradzki <mgradzki@cisco.com>2018-03-07 14:51:18 +0100
committerMarek Gradzki <mgradzki@cisco.com>2018-03-07 14:53:33 +0100
commit9659d28f1083f8ce9c67db3fb708f5b96740fba4 (patch)
treec27be4f85bdd4cd626abcfc5ed7b0228cfc78e97 /infra/impl
parenta604b493ab32467635ec1767b2747bcf8f22cae9 (diff)
Move shutdown hook registration to dedicated provider
This change would allow using ShutdownHandlerImpl in integration tests. Change-Id: I854f65d03b9e1a495fda7211fd1b7fd51b4e5ba3 Signed-off-by: Marek Gradzki <mgradzki@cisco.com>
Diffstat (limited to 'infra/impl')
-rw-r--r--infra/impl/src/main/java/io/fd/honeycomb/impl/ShutdownHandlerImpl.java6
1 files changed, 2 insertions, 4 deletions
diff --git a/infra/impl/src/main/java/io/fd/honeycomb/impl/ShutdownHandlerImpl.java b/infra/impl/src/main/java/io/fd/honeycomb/impl/ShutdownHandlerImpl.java
index 71145817f..f622a603b 100644
--- a/infra/impl/src/main/java/io/fd/honeycomb/impl/ShutdownHandlerImpl.java
+++ b/infra/impl/src/main/java/io/fd/honeycomb/impl/ShutdownHandlerImpl.java
@@ -16,7 +16,6 @@
package io.fd.honeycomb.impl;
-import com.google.common.annotations.VisibleForTesting;
import io.fd.honeycomb.data.init.ShutdownHandler;
import java.util.Deque;
import java.util.LinkedList;
@@ -32,7 +31,6 @@ public final class ShutdownHandlerImpl implements ShutdownHandler {
public ShutdownHandlerImpl() {
components = new LinkedList<>();
- Runtime.getRuntime().addShutdownHook(new Thread((this::performShutdown)));
}
@Override
@@ -60,8 +58,8 @@ public final class ShutdownHandlerImpl implements ShutdownHandler {
}
}
- @VisibleForTesting
- void performShutdown() {
+ @Override
+ public void performShutdown() {
// close components in reverse order that they were registered
components.descendingIterator().forEachRemaining(closeable -> {
LOG.info("Closing component {}", closeable.getName());