goutamadwant commented on PR #11123: URL: https://github.com/apache/seatunnel/pull/11123#issuecomment-4748249692
@DanielLeens Updated this based on the review The GTID option is no longer treated as a replacement for file/pos. For specific startup, file and pos stay mandatory because the Debezium offset loader still requires them. gtid-set and skip fields are now added on top of that loader-compatible offset map. Also added a regression test that loads the generated offset through MySqlOffsetContext.Loader, updated the GTID e2e config to pass file/pos/gtid/skip together, and made the earliest-offset e2e test clean source/sink state before starting so it should not depend on test order. Verified locally with: ``` ./mvnw -pl seatunnel-connectors-v2/connector-cdc/connector-cdc-mysql -Dtest=MySqlIncrementalSourceStartupConfigTest test ./mvnw -pl seatunnel-e2e/seatunnel-connector-v2-e2e/connector-cdc-mysql-e2e -DskipTests test-compile ./mvnw -pl seatunnel-connectors-v2/connector-cdc/connector-cdc-mysql -DfailIfNoTests=false package ``` -- 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]
