avnijainnn commented on PR #12607: URL: https://github.com/apache/seatunnel/pull/12607#issuecomment-5989260643
Thanks — I went with option (b) to keep this PR within the scope confirmed in [#11007](https://github.com/apache/seatunnel/issues/11007#issuecomment-5917734796). I’ve updated the PR description with a scope note clarifying that this change intentionally performs whole-string nonblank validation only. Per-entry validation/trimming of comma-separated master addresses would change the existing runtime parsing semantics, so I’m leaving that for a separate follow-up. I also adjusted the user-facing wording so it no longer implies that whitespace-padded individual entries are guaranteed to be valid at runtime. -- 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]
