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

    https://github.com/apache/storm/pull/2345#discussion_r143499778
  
    --- Diff: 
storm-server/src/main/java/org/apache/storm/daemon/supervisor/Slot.java ---
    @@ -104,9 +111,12 @@
                 this.port = port;
                 this.iSupervisor = iSupervisor;
                 this.localState = localState;
    +            this.changingCallback = changingCallback;
             }
         }
    -    
    +
    +    //TODO go through all of the state transitions and make sure we handle 
changingBlobs
    +    //TODO make sure to add in transition helpers that clean changingBlobs 
&& pendingChangeingBlobs for not the current topology
    --- End diff --
    
    ??


---

Reply via email to