Github user zd-project commented on a diff in the pull request:
https://github.com/apache/storm/pull/2754#discussion_r209037387
--- Diff:
storm-webapp/src/main/java/org/apache/storm/daemon/logviewer/utils/DirectoryCleaner.java
---
@@ -186,7 +192,22 @@ private boolean isFileEligibleToSkipDelete(boolean
forPerDir, Set<String> active
break;
}
}
+ } catch (IOException e) {
+ ExceptionMeters.NUM_FILE_OPEN_EXCEPTIONS.mark();
--- End diff --
scanned would be the same as read---
