diff options
author | Harald Musum <musum@oath.com> | 2018-06-07 14:47:09 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2018-06-07 14:47:09 +0200 |
commit | 6d06cec92ab856c340e69a30cf0d11bf2972cd6b (patch) | |
tree | 8ad7877f5c0c6c5ad528dbd48c70a74a76c31c87 /node-admin | |
parent | 58ee04c484f89da0a0043a33c74704d87e6dc8d3 (diff) | |
parent | db945d262c87c43af194275ff1d049b2b659d004 (diff) |
Merge pull request #6117 from vespa-engine/freva/always-close-files-list
Always close files list
Diffstat (limited to 'node-admin')
-rw-r--r-- | node-admin/src/main/java/com/yahoo/vespa/hosted/node/admin/maintenance/StorageMaintainer.java | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/node-admin/src/main/java/com/yahoo/vespa/hosted/node/admin/maintenance/StorageMaintainer.java b/node-admin/src/main/java/com/yahoo/vespa/hosted/node/admin/maintenance/StorageMaintainer.java index 7f2d1f1eff7..a7bf22591d4 100644 --- a/node-admin/src/main/java/com/yahoo/vespa/hosted/node/admin/maintenance/StorageMaintainer.java +++ b/node-admin/src/main/java/com/yahoo/vespa/hosted/node/admin/maintenance/StorageMaintainer.java @@ -276,8 +276,8 @@ public class StorageMaintainer { */ public void handleCoreDumpsForContainer(ContainerName containerName, NodeSpec node, boolean force) { // Sample number of coredumps on the host - try { - numberOfCoredumpsOnHost.sample(Files.list(environment.pathInNodeAdminToDoneCoredumps()).count()); + try (Stream<Path> files = Files.list(environment.pathInNodeAdminToDoneCoredumps())) { + numberOfCoredumpsOnHost.sample(files.count()); } catch (IOException e) { // Ignore for now - this is either test or a misconfiguration } |