lostluck commented on issue #31658: URL: https://github.com/apache/beam/issues/31658#issuecomment-2631621616
The first one is incorrect, and should be a failure at construction time since two triggers are being specified. The second time, I would expect it to work though, but triggers with merging Windows are pretty complicated... I agree that it doesn't match with my intuition which would be the merge is supposed to take the earlier of the two "first element times", rather than the later of them... Which does appear to be the implementation, but it's possible the timer that is being set isn't mapping back to the newly merged window... (which doesn't seem right). Hard to say offhand unless we validate the equivalent outside of Go. There could be some other issue. -- 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]
