flink-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From uce <...@git.apache.org>
Subject [GitHub] flink pull request #5331: [FLINK-8473][webUI] Improve error behavior of JarL...
Date Mon, 22 Jan 2018 13:29:19 GMT
Github user uce commented on a diff in the pull request:

    https://github.com/apache/flink/pull/5331#discussion_r162935682
  
    --- Diff: flink-runtime-web/src/main/java/org/apache/flink/runtime/webmonitor/handlers/JarListHandler.java
---
    @@ -77,6 +82,11 @@ public boolean accept(File dir, String name) {
     						}
     					});
     
    +					if (list == null) {
    +						LOG.warn("Jar storage directory {} has been deleted.", jarDir.getAbsolutePath());
    --- End diff --
    
    Somehow this comment got lost before: Should we make the error message more explicit and
say that it was `deleted externally` (e.g. not by Flink)?


---

Mime
View raw message