GitHub user jerrypeng opened a pull request:

    https://github.com/apache/storm/pull/2314

    [STORM-2731] - Simple checks in Storm Windowing

    There is also inconsistent and mixed used of Longs and Ints throughout the 
windowing code.  Perhaps we should just change all the numeric values to use 
longs?  Especially since all calculation of time is done in milliseconds.  It 
would also be nice to make clear in both the code and documentation that all 
timestamps and time based calculations is in milliseconds.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/jerrypeng/storm STORM-2731

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/storm/pull/2314.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #2314
    
----
commit 40d133b6c9f434eabc3b50bb8f4ebf91d5999930
Author: Jerry Peng <jerrypeng@jerrys-macbook-pro.local>
Date:   2017-09-07T22:32:39Z

    [STORM-2731] - Simple checks in Storm Windowing

----


---

Reply via email to