SEZ9 commented on issue #11007: URL: https://github.com/apache/seatunnel/issues/11007#issuecomment-5862315907
@karthik-0306 thanks for volunteering. Nobody else in this thread has claimed `connector-paimon`, so it is open for you to take, subject to scope confirmation. Before opening a PR, please post a short scope note here so we can confirm it: - Which Paimon Source/Sink factory `optionRule()` options are currently marked required on `dev`, and which of those lack a `Conditions.notBlank` (or equivalent declarative) check. - Confirmation that the change is limited to declarative validation for those existing required options — no new options, no runtime connectivity checks, and no changes to catalog/client construction. - Planned factory tests covering valid input plus missing, empty, and whitespace-only values for each option you touch. - Whether the EN/ZH option docs already state these values must be nonblank; update them only if they don't. Once the scope is agreed, open one focused PR against `dev` and link it in this thread so the tracker row can be updated. <!-- streview-comment:1366 --> -- 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]
