Skip to content

[Test][E2E] Stabilize JDBC schema change assertions - #11200

Merged
davidzollo merged 10 commits into
apache:devfrom
DanielLeens:dev-jdbc-schema-order-fix-20260627
Jun 30, 2026
Merged

[Test][E2E] Stabilize JDBC schema change assertions#11200
davidzollo merged 10 commits into
apache:devfrom
DanielLeens:dev-jdbc-schema-order-fix-20260627

Conversation

@DanielLeens

@DanielLeens DanielLeens commented Jun 27, 2026

Copy link
Copy Markdown
Contributor

Summary

  • compare JDBC schema-change sink columns by normalized column names instead of physical position
  • project sink rows with the current MySQL source column order before asserting row equality
  • wait for SQL Server recovery and confirm XA procedure installation before exactly-once schema evolution tests continue

Validation

  • ./mvnw -B -o -pl seatunnel-e2e/seatunnel-connector-v2-e2e/connector-jdbc-e2e/connector-jdbc-e2e-ddl -am spotless:apply -DskipTests
  • docker pull testcontainers/ryuk:0.3.4
  • ./mvnw -B -o -pl seatunnel-e2e/seatunnel-connector-v2-e2e/connector-jdbc-e2e/connector-jdbc-e2e-ddl -Dtest=SqlServerSchemaChangeIT -Dsurefire.failIfNoSpecifiedTests=false test
    • SQL Server recovery and sp_sqljdbc_xa_install completed successfully
    • full SeaTunnel E2E execution was then blocked locally because host port 5801 was already occupied by an existing standalone SeaTunnelServer process

@davidzollo
davidzollo force-pushed the dev-jdbc-schema-order-fix-20260627 branch from e71dcfc to 051b723 Compare June 29, 2026 13:33

@nzw921rx nzw921rx left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1 LGTM

@davidzollo davidzollo left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@davidzollo
davidzollo merged commit 1827600 into apache:dev Jun 30, 2026
3 checks passed
junjunclub pushed a commit to junjunclub/seatunnel that referenced this pull request Jul 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants