diff options
author | Valerij Fredriksen <valerijf@oath.com> | 2018-10-12 13:52:00 +0200 |
---|---|---|
committer | Valerij Fredriksen <valerijf@oath.com> | 2018-10-12 13:52:00 +0200 |
commit | 976c2294b0cd9460c89d40bbb586af944c7edc25 (patch) | |
tree | fc81e9c13f8ff6ed19ccf3478e0477bd742848cd /node-admin | |
parent | 614ec88a2d210150ae0a93d81a5931917695f5a4 (diff) |
Fixes
Diffstat (limited to 'node-admin')
2 files changed, 2 insertions, 5 deletions
diff --git a/node-admin/src/main/java/com/yahoo/vespa/hosted/node/admin/maintenance/coredump/CoredumpHandler.java b/node-admin/src/main/java/com/yahoo/vespa/hosted/node/admin/maintenance/coredump/CoredumpHandler.java index 1b4ac09d231..16037f3233b 100644 --- a/node-admin/src/main/java/com/yahoo/vespa/hosted/node/admin/maintenance/coredump/CoredumpHandler.java +++ b/node-admin/src/main/java/com/yahoo/vespa/hosted/node/admin/maintenance/coredump/CoredumpHandler.java @@ -47,10 +47,6 @@ public class CoredumpHandler { private final Supplier<String> coredumpIdSupplier; /** - * - * @param terminal - * @param coreCollector - * @param coredumpReporter * @param crashPathInContainer path inside the container where core dump are dumped * @param doneCoredumpsPath path on host where processed core dumps are stored */ @@ -150,7 +146,7 @@ public class CoredumpHandler { } /** - * Compresses core file (and deletes the uncompressed cored), then moves the entire core dump processing + * Compresses core file (and deletes the uncompressed core), then moves the entire core dump processing * directory to {@link #doneCoredumpsPath} for archive */ private void finishProcessing(NodeAgentContext context, Path coredumpDirectory) throws IOException { diff --git a/node-admin/src/test/java/com/yahoo/vespa/hosted/node/admin/maintenance/coredump/CoredumpHandlerTest.java b/node-admin/src/test/java/com/yahoo/vespa/hosted/node/admin/maintenance/coredump/CoredumpHandlerTest.java index 77f32d1933a..d567cac9a70 100644 --- a/node-admin/src/test/java/com/yahoo/vespa/hosted/node/admin/maintenance/coredump/CoredumpHandlerTest.java +++ b/node-admin/src/test/java/com/yahoo/vespa/hosted/node/admin/maintenance/coredump/CoredumpHandlerTest.java @@ -54,6 +54,7 @@ public class CoredumpHandlerTest { private final TestTerminal terminal = new TestTerminal(); private final CoreCollector coreCollector = mock(CoreCollector.class); private final CoredumpReporter coredumpReporter = mock(CoredumpReporter.class); + @SuppressWarnings("unchecked") private final Supplier<String> coredumpIdSupplier = mock(Supplier.class); private final CoredumpHandler coredumpHandler = new CoredumpHandler(terminal, coreCollector, coredumpReporter, crashPathInContainer, doneCoredumpsPath, coredumpIdSupplier); |