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.
---