You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@storm.apache.org by srdo <gi...@git.apache.org> on 2017/09/19 20:43:45 UTC

[GitHub] storm issue #2314: [STORM-2731] - Simple checks in Storm Windowing

Github user srdo commented on the issue:

    https://github.com/apache/storm/pull/2314
  
    I agree, we should use long for time unit parameters. 
    
    A decent way to make clear that timestamps are in milliseconds is to name the relevant variables so it is obvious, e.g. `timeMs` instead of `time`.


---