yigitcan-ozturk opened a new pull request, #12174:
URL: https://github.com/apache/seatunnel/pull/12174
### Purpose of this pull request
Related to #11007.
This PR implements the accepted connector-datahub factory-validation slice.
### What changed
- Keep `endpoint`, `accessId`, `accessKey`, `project`, and `topic` required.
- Add declarative `Conditions.notBlank(...)` validation to each required
string option.
- Replace the placeholder factory test with focused validation coverage for:
- valid configuration;
- missing required options;
- empty required options;
- whitespace-only required options.
- Preserve option names and all existing nonblank configurations.
- Leave client construction, timeout/retry semantics, network behavior, and
runtime error handling unchanged.
### Does this PR introduce _any_ user-facing change?
Yes.
Empty or whitespace-only values for required DataHub sink connection options
are now rejected during connector option validation instead of reaching the
DataHub client/runtime path.
Existing nonblank configurations are unchanged.
### How was this patch tested?
Focused `ConfigValidator` tests cover:
- valid configuration;
- missing required options;
- empty required options;
- whitespace-only required options;
for all five required string options.
Repository CI will run the connector test suite for this PR.
### Check list
* [ ] If any new Jar binary package adding in your PR, please add License
Notice according
[New License
Guide](https://github.com/apache/seatunnel/blob/dev/docs/en/developer/new-license.md)
* [ ] If necessary, please update the documentation to describe the new
feature.
* [ ] If necessary, please update `incompatible-changes.md`.
--
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]
