arunpandianp commented on PR #33037: URL: https://github.com/apache/beam/pull/33037#issuecomment-2461321438
This fixes https://github.com/apache/beam/issues/20203. I think this is a safe change. Not 100% sure if an element with timestamp `BoundedWindow.TIMESTAMP_MAX_VALUE` and the cleanup timer can race. I don't think they will, since timer will be fired only after the element is processed and watermark becomes > `BoundedWindow.TIMESTAMP_MAX_VALUE` -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
