SEZ9 commented on issue #10681: URL: https://github.com/apache/seatunnel/issues/10681#issuecomment-5787791064
@junjunclub thanks for the updated Layer 1 matrix. I'm only commenting on the Redis `--dry-run connect` slice here. The Redis Source / Sink PR is listed as in progress, but its branch currently has no implementation diff, so please keep Redis described as planned rather than implemented in the matrix until the validator, tests, documentation and E2E commits are pushed. Once the PR has a real diff, updating the support list is the right next step. For the Redis work itself, the earlier asks still stand: - One focused source/sink PR built only on `SupportSourceDryRunValidation` and `SupportSinkDryRunValidation`; no changes to `DryRunConnectValidator` or the shared SPI. - Bounded connectivity/authentication validation for both standalone and cluster modes, closing every client on success and on failure. - No reads, key scans, writes, or any mutation of server state during validation. - Source schema taken from the configured schema/table configuration, not derived from Redis values. - Focused success and authentication/connection-failure tests, plus matching English and Chinese connector documentation. - Link the PR status back here when it is ready for review. @goutamadwant noted, thanks for leaving Redis to junjunclub's planned work to avoid overlap; nothing further needed from you on this slice. <!-- streview-comment:1243 --> -- 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]
