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

   ### Purpose of this pull request
   
   Closes #11899.
   
   `TransformErrorToMysqlIT` and `SinkErrorToMysqlIT` currently download MySQL 
Connector/J from Maven Central with `curl` inside the SeaTunnel test container. 
This change uses the existing E2E `DependencyJar` helper to copy the 
already-declared MySQL driver from the test classpath into 
`/tmp/seatunnel/plugins/Jdbc/lib`.
   
   This removes the container-side network and `curl` dependency while keeping 
the existing MySQL Connector/J dependency and version unchanged.
   
   AI assistance disclosure: OpenAI Codex (GPT-5) assisted with implementation 
and verification of this change.
   
   ### Does this PR introduce _any_ user-facing change?
   
   No.
   
   ### How was this patch tested?
   
   - `./mvnw spotless:apply`
   - `./mvnw -T 1 -B verify -Dapi.version=1.40 -DskipUT=true -DskipIT=false 
-Dit.test=TransformErrorToMysqlIT,SinkErrorToMysqlIT 
-Dit.failIfNoSpecifiedTests=false -DfailIfNoTests=false 
-Dlicense.skipAddThirdParty=true -Dskip.ui=true --no-snapshot-updates -pl 
:connector-seatunnel-e2e-base -am -Pci`
     - Result: 31 tests run, 0 failures, 0 errors, 2 skipped.
     - The submitted jobs loaded 
`file:/tmp/seatunnel/plugins/Jdbc/lib/mysql-connector-j-8.0.32.jar`.
   - `./mvnw -q -DskipTests verify`
   - `git diff --check`
   
   ### Check list
   
   * [x] No new Jar binary package is added.
   * [x] Documentation is not necessary because there is no user-facing change.
   * [x] No incompatible change is introduced.
   * [x] The connector checklist is not applicable because this only changes 
E2E dependency setup.


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