Nikk8091 commented on issue #11007: URL: https://github.com/apache/seatunnel/issues/11007#issuecomment-5668323157
Confirmed. I will implement the corrected SensorsData contract exactly as requested: `server_url` will be registered as required with a nonblank constraint, while `consumer` will be registered as optional and retain its existing `batch` default and both console/batch modes. The PR will contain only the `SensorsDataSDKSinkFactory` option registration/validation change, focused factory tests for valid, missing/empty/whitespace-only `server_url` and omitted `consumer`, and matching EN/ZH documentation. I will not add format, SDK, credential, URL-format, numeric, or remote checks, and runtime behavior will remain unchanged. -- 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]
