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 ---- ---