aljoscha commented on pull request #12640: URL: https://github.com/apache/flink/pull/12640#issuecomment-648203540
Sorry for the wait! I think this change is also good. I thought about another thing though: for event-time it could make sense to not modify the window that an element is in, i.e. the staggering doesn't change the actual windows. We could only change the firing timestamp, meaning we would change the trigger we set in `EventTimeTrigger` based on staggering. Potentially we would move the firing timestamp into the future, but never before the original firing timestamp. Did you ever think about this? ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected]
