DanielLeens opened a new pull request, #11137:
URL: https://github.com/apache/seatunnel/pull/11137
## What's changed
- switch the SQL Server container setup to the supported `MSSQL_SA_PASSWORD`
variable and configure the MSDTC RPC/DTC ports required by containerized XA
transactions
- wait for a real `sqlcmd` login before running XA setup so the test does
not race SQL Server upgrade work
- fail fast when `sp_configure` or `sp_sqljdbc_xa_install` fails, and wait
until `xp_sqljdbc_xa_init_ex` becomes visible before the exactly-once path
starts
- remove the invalid `external user enabled` setup step and relax the
ready-log matcher so startup does not depend on a trailing newline
## Validation
- `./mvnw -nsu -pl
seatunnel-e2e/seatunnel-connector-v2-e2e/connector-jdbc-e2e/connector-jdbc-e2e-ddl
spotless:apply`
- `./mvnw -nsu -pl
seatunnel-e2e/seatunnel-connector-v2-e2e/connector-jdbc-e2e/connector-jdbc-e2e-ddl
-q -DskipTests verify`
- `./mvnw -nsu -pl
seatunnel-e2e/seatunnel-connector-v2-e2e/connector-jdbc-e2e/connector-jdbc-e2e-ddl
-DfailIfNoTests=false -Dtest=SqlServerSchemaChangeIT test`
- on
`/Volumes/disk2/work/myworkspace/tmp/seatunnel-dev-sqlserver-xa-init-20260620`,
the host filesystem exposed `src/test/resources/docker/setup.sql` to the MySQL
container with unreadable permissions and the run stopped before the SQL Server
path (`Permission denied`)
- on `/Users/stone/work/myworkspace/seatunnel_third` with normal file
permissions, MySQL initialization succeeded and SQL Server 2022 still failed
under the local Docker Desktop memory cap before reaching the ready log
(`Available Physical Memory 89.32 mb`, `Error 701`, `Failed allocate pages`)
- `seatunnel-engine-ui` was skipped because this change does not touch that
module
- no modules or deployment artifacts were reused in this run
## References
-
https://learn.microsoft.com/en-us/sql/linux/containers/configure-distributed-transactions?view=sql-server-ver17
-
https://learn.microsoft.com/en-us/sql/connect/jdbc/understanding-xa-transactions?view=sql-server-ver17
-
https://learn.microsoft.com/el-gr/sql/linux/configure/environment-variables?view=sql-server-linux-ver17
--
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]
