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