Github user aljoscha commented on the issue: https://github.com/apache/flink/pull/5295 I think the changes are good! Thanks for working on this. ð As a final change before merging, I would annotate the new classes/methods as `@PublicEvolving`, would you be ok with that? And I would also like to change `SessionWindowTimeGapExtractor.extract()` to return a long instead of `Time`. What do you think?