Github user srdo commented on the issue:

    https://github.com/apache/storm/pull/2743
  
    As I see it there are still some issues that were commented on but are not 
resolved.
    
    * The TODOs should be removed/fixed
    * Comment here about adding a comment 
https://github.com/apache/storm/pull/2743/files#r199600941
    * Comment here about making stopTiming idempotent 
https://github.com/apache/storm/pull/2743/files#r199531108
    
    Other than that, this looks pretty good to me.


---

Reply via email to