DanielLeens commented on issue #11007: URL: https://github.com/apache/seatunnel/issues/11007#issuecomment-5635796874
I rechecked the current `dev` factories and the complete open PR set before deciding the pending claims. I found no matching open validation PR for the three scopes below. No labels or formal assignees were changed. - @yigitcan-ozturk: BigQuery sink is available. `project_id`, `dataset_id`, and `table_id` are already required but do not have declarative blank checks. Limit the PR to nonblank validation and focused valid/missing/empty/whitespace cases for those three options. Preserve authentication, write mode, sequence, batching, emulator and remote behavior. - @1328837476-hug: GoogleFirestore sink is available. Keep `project_id` and `collection` required and nonblank; keep `credentials` optional, but reject an explicitly blank value. Do not add credential parsing, format, remote, or SDK checks. - @Nikk8091: SensorsData is available with the corrected factory name, `SensorsDataSDKSinkFactory`. `server_url` has no default and is consumed directly, so make it required and nonblank. Keep `consumer` optional with its existing `batch` default and preserve both consumer modes. No format, SDK, or remote checks. Each slice needs one connector-scoped PR, factory-level regression coverage for the stated contract, and matching EN/ZH documentation. Rebase immediately before opening the PR and link it here; do not combine any of these connectors or change runtime behavior. -- 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]
