davidzollo commented on PR #10925: URL: https://github.com/apache/seatunnel/pull/10925#issuecomment-5139527294
I rechecked the active unresolved review threads against the current head `53efd4a16840d236cc2c77cd10d65e2db1ffe301`. Addressed items: - Kafka exactly-once streaming config now documents why the readiness probe uses `start_mode = earliest`, a fresh `consumer_group`, and why no preexisting `__consumer_offsets` are expected before the job restores checkpointed splits. - The Kafka exactly-once polling path now has regression coverage for continuing an empty `read_committed` poll when the partition position advances. - The TDengine single-vgroup CI fallback now has an explanatory comment, and the readiness setup avoids relying on fragile repeated create/drop behavior. - FTP/SFTP/RocketMQ helper comments were added where ownership/lifecycle and unique-test suffix behavior needed reviewer context. - The MySQL CDC schema-change timestamp drift window has been tightened while keeping the Awaitility timeout as the outer CI failure deadline. The latest fork Build for this head is queued: https://github.com/davidzollo/seatunnel/actions/runs/30593748386 . Please re-review/resolve the addressed threads when convenient. -- 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]
