DanielLeens commented on issue #11007: URL: https://github.com/apache/seatunnel/issues/11007#issuecomment-5715973562
Thanks for surfacing the overlap. **Classification: F / Connector-V2 configuration-validation tracking.** The SensorsData slice is already owned by #12315: it changes `server_url` and `consumer` in the same factory, includes the focused missing/empty/whitespace plus batch/console/default coverage, targets `dev`, and its current Build is successful. #12367 overlaps that factory and repeats the `server_url` change, while adding `bulk_size` and `max_cache_row_size` without equivalent focused coverage; its Build is also `ACTION_REQUIRED`. It must not proceed as a second SensorsData implementation or define the tracker contract. @TianHengZhuang, please keep any follow-up for the existing SensorsData scope on #12315 rather than adding more work to #12367. @nzw921rx, I could not find a `connector-cake` module or an open Cake validation PR on current `dev`; please provide the exact module/path and the intended imperative validation before redirecting this contributor. No separate claim is confirmed until that evidence is available. -- 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]
