DanielLeens commented on issue #11007: URL: https://github.com/apache/seatunnel/issues/11007#issuecomment-5559104569
Thanks @yigitcan-ozturk and @xinnyuli. I checked current dev, the tracker, and the open upstream PR set. The Sentry slice is available. SentrySinkFactory currently requires dsn for presence only, so the proposed declarative nonblank constraint and focused factory-validation coverage are a good fit. This is an accepted connector-sentry-only claim. The Slack slice is also available. SlackSinkFactory currently requires webhooks_url, oauth_token, and slack_channel for presence only. @xinnyuli posted the first complete Slack plan in this thread, so that claim is recorded for @xinnyuli. @1328837476-hug, please select another unclaimed connector so parallel work does not duplicate the same factory change. For each PR, keep the scope to one connector, express only factory-verifiable blank-value rules through the existing declarative condition, and add valid, empty, and whitespace-only validation cases. Preserve option names, existing nonblank configurations, and runtime network behavior. Do not modify the writer/client path or add remote validation. Please link the focused PR here when it is ready. -- 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]
