abolfazlmadanii opened a new pull request, #11817:
URL: https://github.com/apache/seatunnel/pull/11817

   ### Purpose of this pull request
   
   Part of #11007.
   
   This PR completes the Email sink declarative option rule by:
   
   - declaring the SMTP port, attachment name, and field delimiter options;
   - validating that email_smtp_port is greater than zero with 
Conditions.greaterThan;
   - adding focused tests for valid, default, zero, and negative SMTP port 
values;
   - documenting the SMTP port constraint in English and Chinese.
   
   ### Does this PR introduce _any_ user-facing change?
   
   Yes. Invalid non-positive email_smtp_port values are now rejected during 
option validation. The default value and valid configurations remain unchanged.
   
   ### How was this patch tested?
   
   - ./mvnw -pl seatunnel-connectors-v2/connector-email test
   - ./mvnw spotless:apply
   - ./mvnw -q -DskipTests verify
   
   The connector-email tests passed with 5 tests and no failures. The full 
./mvnw test command was also attempted, but the reactor stopped in the 
unrelated seatunnel-config-shade module because shaded ConfigFactory classes 
were unavailable in its test classpath.
   
   ### Check list
   
   * [x] No new Jar binary package is added.
   * [x] English and Chinese Email connector documentation is updated.
   * [x] No incompatible-change entry is required; valid configurations and 
defaults are unchanged.
   * [x] Existing connector packaging, mapping, labels, and E2E wiring are 
unchanged.


-- 
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]

Reply via email to