DanielLeens commented on issue #10899: URL: https://github.com/apache/seatunnel/issues/10899#issuecomment-4608236002
Thanks for adding the regression test. That is the missing proof for this path. Keeping the restored split's concrete binlog offset instead of falling back to the original timestamp bootstrap is the right behavior, and the extra test makes the fix much safer. From the issue side, the path now looks well covered: root cause, fix direction, and regression protection are all in place. Since PR #10987 is still open, I think this issue should stay open until that PR is merged, but the next step is mainly review / merge rather than more root-cause analysis here. -- 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]
