Github user agresch commented on a diff in the pull request:
https://github.com/apache/storm/pull/2925#discussion_r240360980
--- Diff:
storm-server/src/main/java/org/apache/storm/blobstore/BlobStoreUtils.java ---
@@ -204,6 +204,14 @@ public static boolean downloadUpdatedBlob(Map<String,
Object> conf, BlobStore bl
} catch (Exception exp) {
// Logging an exception while client is connecting
LOG.error("Exception", exp);
+ } finally {
+ if (null != out) {
--- End diff --
same here, shouldn't we have nulled out out on close?
---
