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

    https://github.com/apache/storm/pull/2502#discussion_r167287026
  
    --- Diff: storm-client/src/jvm/org/apache/storm/StormTimer.java ---
    @@ -97,6 +97,8 @@ public void run() {
                             // events.
                             Time.sleep(1000);
                         }
    +                    if(Thread.interrupted())
    +                        this.active.set(false);
    --- End diff --
    
    Nit can we wrap this in `'{'` and `'}'` and have a space after the `if` to 
match the style guide.


---

Reply via email to