DanielLeens commented on issue #11007: URL: https://github.com/apache/seatunnel/issues/11007#issuecomment-5917734796
Classification: I / Declarative OptionRule migration tracking. I checked the three proposed slices against current `dev` (`53733a112c9e935c5d97f7f24be249caa43b4fd5`) and the open PR set. They are distinct and available: * @Nikk8091: `connector-socket` Source only. Keep the scope to declarative nonblank `host` and positive `port` validation. Do not change address parsing, defaults, framing, or the Sink path. * @yigitcan-ozturk: ClickHouse Source `host` only. Do not include the separate ClickHouse File delimiter work or broaden this into batch/split configuration validation. * @avnijainnn: Kudu `kudu_masters` only in the Source, Sink, and Catalog factories. Preserve the existing address parsing, Kerberos behavior, table-selection rules, and configuration values; validation must not trim or normalize a valid value. Please keep these as three focused PRs, each with the relevant existing factory tests proving missing, empty, whitespace-only, and valid values. For each PR, show that removing the new declarative rule makes the corresponding regression test fail. Do not combine unrelated connector migrations under this umbrella. -- 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]
