scwhittle commented on pull request #13472: URL: https://github.com/apache/beam/pull/13472#issuecomment-737949989
R: @aromanenko-dev I see you added this with #5019, would you be able to review? In particular, do you think using kafka create time for nexmark event timestamps should be the new default (for reading/writing) or just optional? Without these changes populating a topic by running query 0 and then reading the topic for other queries has incorrect watermark semantics, given that nexmark uses historical events by default. @reuvenlax FYI ---------------------------------------------------------------- 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]
