[GitHub] storm pull request #2309: (1.x) STORM-2724 Shutdown ExecutorService in Water...

[srdo]

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

    https://github.com/apache/storm/pull/2309#discussion_r137560854
  
    --- Diff: 
storm-core/test/jvm/org/apache/storm/windowing/WaterMarkEventGeneratorTest.java 
---
    @@ -56,6 +57,13 @@ public void add(Event<Integer> event) {
             waterMarkEventGenerator.start();
         }
     
    +    @After
    +    public void tearDown() {
    +        if (waterMarkEventGenerator != null) {
    --- End diff --
    
    Nit: I don't think this can be null


---