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

    https://github.com/apache/storm/pull/2618#discussion_r178693115
  
    --- Diff: 
storm-server/src/main/java/org/apache/storm/localizer/LocalizedResourceRetentionSet.java
 ---
    @@ -94,17 +93,17 @@ public void cleanup(ClientBlobStore store) {
                 Map.Entry<LocallyCachedBlob, Map<String, ? extends 
LocallyCachedBlob>> rsrc = i.next();
                 LocallyCachedBlob resource = rsrc.getKey();
                 try {
    -                resource.getRemoteVersion(store);
    +                if (!store.isRemoteBlobExists(resource.getKey())) {
    --- End diff --
    
    Ok, i will separate it apart with another JIRA.


---

Reply via email to