[STORM-2747] Make the windowing classes use long instead of int for time parameters, and rename millisecond based time parameters so it's clear which unit they are.

• Type: Improvement
• Status: Open
• Priority: Minor
• Resolution: Unresolved
• Affects Version/s: 2.0.0, 1.1.1, 1.0.5
• Assignee: Unassigned
• Reporter: Stig Rohde Døssing
• Created: 20/Sep/17 07:49
• Updated: 20/Sep/17 07:49

Description

Some parameters in the windowing classes are using int instead of long for describing time, which requires a bunch of casting to/from long in the code. Some variables are always in milliseconds but are named e.g. "value", which makes it easy to accidentally use the wrong timeunit. Such variables should be named e.g. "valueMs" to disambiguate the unit.