je-ik commented on pull request #14718: URL: https://github.com/apache/beam/pull/14718#issuecomment-847215896
If we enforce the use of explicit output timestamp I think that from the correctness perspective, it really looks like c) is the way to go. I'm still not convinced we can simply ignore the fact that it is breaking change, though. Mostly because it is a *runtime* breaking change, that can result in production downtimes (if a buggy code is not revealed during testing of a pipeline, which might happen). -- 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]
