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

    https://github.com/apache/storm/pull/2764#discussion_r209336348
  
    --- Diff: 
storm-server/src/main/java/org/apache/storm/daemon/nimbus/Nimbus.java ---
    @@ -2905,6 +2929,10 @@ public void launchServer() throws Exception {
                                                 throw new RuntimeException(e);
                                             }
                                         });
    +
    +            //Should we make the delaySecs and recurSecs in sync with any 
conf value?
    --- End diff --
    
    Okay, we can do it later. Filed 
https://issues.apache.org/jira/browse/STORM-3192. Please remove the comment 
since it seems like it wouldn't be useful outside this PR. 


---

Reply via email to