Github user arunmahadevan commented on the issue:

    https://github.com/apache/storm/pull/1527
  
    @darionyaphet You need to make more changes,
    
    1. you should use `longValue()` instead of the `intValue()` in the 
`initWindowManager` where the values are read from storm conf.
    2. The corresponding constants defined in the Config classes like 
TOPOLOGY_BOLTS_WINDOW_LENGTH_DURATION_MS are validated for integer range since 
its annotated with ` @isInteger`. This needs to be removed and appropriate 
annotation should be specified.
    
    Its also good to add some unit tests to validate the changes.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to