DanielLeens commented on PR #11569: URL: https://github.com/apache/seatunnel/pull/11569#issuecomment-5381190204
Addressed JeremyXin's two review points on the new head `0d9f281e054a07190feb664e5130de958268b737`: 1. Added a bounded 1s pause between synchronous XA retry rounds in both `commitXidInfos(...)` and `recoverCheckpointTransactions(...)`, so transient `XAER_RMFAIL` / `XA_RETRY` outages do not burn the whole `max_commit_attempts` budget immediately. 2. Re-enabled `XaGroupOpsImplIT` and added a real-MySQL integration test that prepares an XA transaction, stops the resource manager, and verifies the resulting commit failure now propagates back out through `JdbcSinkAggregatedCommitter.commit(...)` as `JdbcConnectorException` instead of being silently swallowed. I also updated both `docs/en` and `docs/zh` JDBC sink docs to describe the new synchronous retry pacing. Build is rerunning on this head now. -- 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]
