DanielLeens commented on issue #10681: URL: https://github.com/apache/seatunnel/issues/10681#issuecomment-5776199438
Classification: B / Zeta progressive dry-run Layer 1 rollout. Thanks for keeping the connector-support matrix current. Current `dev` still uses the opt-in `SupportSourceDryRunValidation` and `SupportSinkDryRunValidation` boundary, so the listed connector slices should continue to reuse that SPI and leave `DryRunConnectValidator` unchanged. I rechecked the linked work. #12399 and #12400 contain connector-specific changes; #12430 is a draft that depends on the Cassandra bootstrap correction in #12429; #12432 and #12433 are active connector slices. #12431 is also correctly marked as a draft, but its current branch has no implementation diff yet, so please describe Redis as planned/draft rather than implemented until the validator, tests, documentation, and E2E commits are pushed. For every slice, keep the user-facing contract explicit: `--dry-run connect` may validate bounded connectivity/authentication and metadata, but it must not read business data, scan, write, initialize a producing transaction, or mutate server state. Please update the matrix only after the corresponding PR has a real diff and link the final PR status back here; later layers and any shared-SPI change should continue to be discussed separately before implementation. -- 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]
