BobSong-dev opened a new pull request, #11831:
URL: https://github.com/apache/seatunnel/pull/11831
Relates to #11007
### Purpose of this pull request
Moves Databend source JDBC URL prefix validation from source construction
into the connector's declarative `OptionRule`. Invalid URLs now fail during
standard configuration validation, and the duplicate runtime check is removed.
### Does this PR introduce _any_ user-facing change?
No. The accepted configuration contract is unchanged; an invalid URL is
rejected earlier by the common configuration validator.
### How was this patch tested?
- Added focused `DatabendFactoryTest` coverage for a valid
`jdbc:databend://` URL and an invalid JDBC URL.
- Ran `mvn spotless:apply` successfully with JDK 8 across the repository.
- Attempted the targeted command below, but the current `dev` checkout fails
before reaching Databend in the unrelated `seatunnel-config-shade` module
because shaded Typesafe Config classes are missing from the local source tree:
```text
mvn -Dskip.spotless=true -pl seatunnel-connectors-v2/connector-databend
-am -Dtest=DatabendFactoryTest -Dsurefire.failIfNoSpecifiedTests=false test
```
Please run the Databend module test in CI/Linux as part of review.
### Check list
* [ ] New JAR binary package
* [ ] Documentation update required
* [ ] Incompatible change
* [x] Existing connector validation migration with focused unit coverage
--
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]
