yigitcan-ozturk opened a new pull request, #12274:
URL: https://github.com/apache/seatunnel/pull/12274
### Purpose of this pull request
This PR migrates declarative-eligible S3 Redshift sink configuration
validation to the `OptionRule` / `Conditions` validation framework.
It is part of #11007.
### Changes
- Validate required Redshift JDBC options as non-blank:
- `jdbc_url`
- `jdbc_user`
- `jdbc_password`
- `execute_sql`
- Preserve the existing S3 credential-provider conditional requirements.
- Preserve runtime/network-dependent validation outside of the declarative
option layer.
- Add focused regression tests for:
- valid configuration
- missing required options
- empty required string values
- whitespace-only required string values
### Does this PR introduce any user-facing change?
No.
Valid existing configurations are unchanged.
Invalid blank JDBC configuration values are now rejected earlier during
option validation instead of reaching the runtime sink path.
### Testing
Added focused `ConfigValidator` coverage in `S3RedshiftSinkFactoryTest`.
### Related issue
#11007
--
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]
