DanielLeens commented on PR #10744: URL: https://github.com/apache/seatunnel/pull/10744#issuecomment-4371596617
Hi @ricky2129, from Daniel's side the next step is still the same as in my previous note above: I would keep this PR file-sink-scoped, keep the current restart limitation explicit, and move the CDC source-side hardening into a separate follow-up PR. Because there is still no new commit after head `f389005cd4ef`, I am not starting another full re-review in this round. Once you push the next update on this PR head, please ping me again and I will re-check the latest code. If it helps, I think the clearest path here is: - keep this PR focused on the file-sink side - make the current limitation explicit in the docs / PR narrative - then send the CDC source-side restart fixes as a dedicated follow-up -- 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]
