Github user kishorvpatil commented on a diff in the pull request:
https://github.com/apache/storm/pull/2871#discussion_r224548982
--- Diff:
storm-server/src/main/java/org/apache/storm/blobstore/BlobStoreUtils.java ---
@@ -191,6 +192,8 @@ public static boolean downloadUpdatedBlob(Map<String,
Object> conf, BlobStore bl
out.close();
}
isSuccess = true;
+ } catch(FileNotFoundException fnf) {
+ LOG.warn("FileNotFoundException", fnf);
--- End diff --
I see. but then, any other `IOException` such as failure to reach Blobstore
etc could cause nimbus restart. I would treat all IOExceptions in same manner.
i.e. log them and return `false`.
---
