DanielLeens commented on PR #10837: URL: https://github.com/apache/seatunnel/pull/10837#issuecomment-4957122265
Thanks for the careful rereview. I traced the same `enable_concurrent_read=false` path again on this unchanged head, and I agree the exactly-once guard bypass is a real blocker here. In the current flow, the early return in `AbstractJdbcSourceChunkSplitter.generateSplits(...)` no longer just changes the split strategy; it also skips the old key-validation path that previously protected `exactly_once=true` tables without a primary / unique key. So from Daniel's side this still needs a code update before merge. Please push the fix, and I will rereview the full latest head again. -- 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]
