You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@storm.apache.org by arunmahadevan <gi...@git.apache.org> on 2018/09/11 19:40:22 UTC

[GitHub] storm pull request #2773: Blobstore sync bug fix

Github user arunmahadevan commented on a diff in the pull request:

    https://github.com/apache/storm/pull/2773#discussion_r216796216
  
    --- 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) {
    --- End diff --
    
    Btw, theres a race condition where the topology gets killed while the blob download is going on and apparently it throws a FNF exception here when invoking out.close().


---