Nikk8091 commented on issue #11007: URL: https://github.com/apache/seatunnel/issues/11007#issuecomment-5582574473
Hi, I would like to request the connector-sls source-and-sink validation slice. I checked the current dev factory rules, tracker comments, and open SLS-related PRs and did not find a matching validation claim or PR; #11779 is documentation work. Both factories currently require endpoint, project, logstore, access_key_id, and access_key_secret for presence only. I propose adding the existing declarative notBlank constraints to these five already-required string options on both factory paths, with focused ConfigValidator tests for valid, individually missing, empty, and whitespace-only values, and acceptance of nonblank values with surrounding spaces. Optional options, defaults, cursor/start-mode behavior, client construction, source/sink runtime behavior, and credential/network checks will remain unchanged. Tests will use dummy credentials and no remote service. This would be one connector-scoped PR with focused EN/ZH documentation updates. Could you confirm availability and scope? This is an additional request, not an assumption of assignment; SensorsData remains on hold and SelectDB Cloud is still awaiting confirmation. Thank you! -- 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]
