DanielLeens commented on issue #10681: URL: https://github.com/apache/seatunnel/issues/10681#issuecomment-5760332358
Thanks for publishing the Layer 1 matrix. I checked current `dev` and found no open Redis `--dry-run connect` PR; Redis also does not currently implement the shared dry-run validation interfaces, so this is a clean next connector slice. Please keep the Redis work in one focused source/sink PR built only on `SupportSourceDryRunValidation` and `SupportSinkDryRunValidation`; do not change `DryRunConnectValidator` or the shared SPI. The validation should establish bounded connectivity/authentication for standalone and cluster modes, close every client on success and failure, and avoid reads, key scans, writes, or mutation. For source schema handling, use configured schema/table configuration rather than Redis values. Please include focused success and authentication/connection-failure tests, plus matching English and Chinese connector documentation, then link the PR back here. The Layer 1 support list can be updated once the PR is open. -- 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]
