danzhewuju commented on PR #4297: URL: https://github.com/apache/flink-cdc/pull/4297#issuecomment-4912024026
> +1. It might be helpful to add a brief code comment explaining the submission-time-based timestamp scenario for future readers. Thank you. I’ve added a comment to clarify the recovery scenario: a TIMESTAMP-startup job may restore using a concrete, SPECIFIC offset; therefore, we should honor the restored split offset rather than reapplying the original submission-time timestamp filter. -- 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]
