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

    https://github.com/apache/storm/pull/2805#discussion_r218107287
  
    --- Diff: 
storm-server/src/main/java/org/apache/storm/daemon/supervisor/Container.java ---
    @@ -219,11 +201,11 @@ public void kill() throws IOException {
                 shutdownTimer = shutdownDuration.time();
             }
             try {
    -            Set<Long> pids = getAllPids();
    +        Set<Long> pids = getAllPids();
     
    -            for (Long pid : pids) {
    -                kill(pid);
    -            }
    +        for (Long pid : pids) {
    +            kill(pid);
    +        }
    --- End diff --
    
    It looks like indentation is off around here.  could be spaces vs tabs or 
something, not really sure.


---

Reply via email to