SEZ9 commented on issue #11007: URL: https://github.com/apache/seatunnel/issues/11007#issuecomment-5806901452
@hutiefang76 Thanks for taking the Fluss sink OptionRule follow-up — the scope as you described it sounds right. To confirm what we'd like to see in the PR: - Add a nonblank condition for the existing sink `bootstrap.servers` option only; keep `database` and `table` optional so multi-table mode keeps working. - Do not add runtime connectivity checks or change client construction. - Include factory tests covering a valid configuration plus missing, empty, and whitespace-only `bootstrap.servers`. - For the EN/ZH docs, update the option description only if it does not already state that this required endpoint must be nonblank. Please open it as one focused PR against `dev` and link it here when ready. No label or assignment change is needed on this issue. <!-- streview-comment:1281 --> -- 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]
