SEZ9 commented on issue #10681: URL: https://github.com/apache/seatunnel/issues/10681#issuecomment-5806335628
@junjunclub thanks for the quick follow-up and for updating the status — much appreciated. For the active connector slices, please keep building on the existing opt-in `SupportSourceDryRunValidation` / `SupportSinkDryRunValidation` boundary and leave `DryRunConnectValidator` and the shared SPI unchanged. Please also keep the `--dry-run connect` contract explicit in each: bounded connectivity/authentication and metadata only — no business-data reads, scans, writes, producing-transaction initialization, or server-state mutation. Move Redis Source / Sink into the in-progress/implemented list only once the validator, tests, documentation, and E2E changes are actually pushed, and link the PR status back here. @goutamadwant thanks for scoping the Cassandra slice to the existing SPI and leaving Redis to the planned work. A short note here once the Cassandra work is ready for review would help keep the matrix accurate. Nothing further needed from me right now — I'll pick this back up as the PRs progress. <!-- streview-comment:1269 --> -- 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]
