SEZ9 commented on issue #11007: URL: https://github.com/apache/seatunnel/issues/11007#issuecomment-6306774388
Thanks @1328837476-hug for carrying the HTTP Sink URL validation slice through: the URL `notBlank` rule, the focused factory tests, and the EN/ZH doc updates are in, and the tracker row for that connector is now marked done. For clarity on what landed: the merged change is intentionally limited to the agreed `notBlank` rule on the existing required URL option plus the factory coverage and docs. It does not change URL parsing, placeholder handling, HTTP execution, retries, or any runtime connectivity check, so there is no follow-up expected on that row. Remaining ask on this point: please do not open a parallel change against the HTTP slice. If you spot a gap in the merged URL validation or its tests, raise it here first so we can scope it before any new PR is opened. <!-- streview-comment:1910 --> -- 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]
