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

   ### Purpose of this pull request
   
   Part of STIP-30 (#11364), tracked by #11354. This is the draft Phase 2A 
validation slice and depends on #11512.
   
   Adds a real MySQL snapshot/binlog-to-JDBC job on an embedded split 
master/worker cluster. The test checks naturally collected reader/enumerator 
reports, snapshot rows, incremental insert/update/delete results, native binlog 
progress, and cleanup after cancellation. It adds no production observation 
endpoint or synthetic reports.
   
   Only three files belong to this follow-up. While #11512 is unmerged, 
GitHub's comparison against `dev` also includes its foundation commits. Review 
the [validation-only 
comparison](https://github.com/goutamadwant/seatunnel/compare/5410c581811304c09954bf2317dfe5c9a2416284...test/stip30-mysql-progress-validation).
 Do not merge this draft before the foundation lands and the diff is reconciled 
against `dev`.
   
   ### Does this PR introduce _any_ user-facing change?
   
   No. This slice adds tests and test-scope dependencies on the existing CDC 
base and MySQL connector modules. No production dependencies, APIs, defaults or 
checkpoint behavior change.
   
   The scenario covers one table and one reader. Source/sink queries verify the 
final DML batch independently; the progress assertion reaches at least one 
record from that batch, not every individual record. Checkpoint/restore fields 
remain explicitly unsupported. Multi-table, heartbeat, recovery and PostgreSQL 
coverage remain follow-ups.
   
   ### How was this patch tested?
   
   - MySQL 8.0.36: the new scenario passed twice on each of Java 8 and Java 11 
across initial and repeat runs.
   - Final clean verification on each JVM selected `MysqlCdcProgressIT` and 
existing `JobExecutionIT#testSayHello`: 2 tests executed, 0 failures, 0 errors, 
0 skipped per JVM.
   - The selected 70-module reactor build passed previously with tests skipped. 
Spotless and whitespace checks passed for this unchanged three-file delta.
   - These are focused local results, not full-suite or cross-platform CI 
completion.
   
   Reproduce from this branch with Java 8 or Java 11 and Docker:
   
   ```sh
   ./mvnw -B \
     -pl seatunnel-e2e/seatunnel-engine-e2e/connector-seatunnel-e2e-base -am \
     verify -DskipUT=true -DskipIT=false \
     '-Dit.test=MysqlCdcProgressIT,JobExecutionIT#testSayHello' \
     -Dfailsafe.failIfNoSpecifiedTests=false -DfailIfNoTests=false \
     -Dskip.ui=true -Dlicense.skipAddThirdParty=true
   ```
   
   Local Docker runs needed the process-only compatibility option 
`'-Dsurefire.jvm.args=-Dfile.encoding=UTF-8 -Dapi.version=1.44'`. No project 
dependency or global Docker configuration was changed.
   
   ### Check list
   
   - [x] Apache headers and formatting checked.
   - [x] No new third-party Jar package or connector registration.
   - [x] No user-facing or incompatible change requiring operational 
documentation.
   - [ ] Complete supported CI and broader engine classpath validation.
   - [ ] Reconcile with merged #11512 and any overlapping test dependency from 
#11947 before marking ready.
   


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