oneby-wang opened a new pull request, #25952: URL: https://github.com/apache/pulsar/pull/25952
Fixes apache/pulsar#25940 ### Motivation `TransactionCoordinatorV5Test.sweepTimeouts_abortsExpiredOpenTxnAndFansOut` intended to verify that the timeout sweeper aborts an expired open transaction and fans out the abort decision to the transaction participant. The test used a 1ms timeout and immediately ran the sweep, assuming the deadline was already in the past. When transaction creation and the sweep happened within the same millisecond, the deadline scan could correctly leave the transaction `OPEN`, making the test flaky. ### Modifications Wait for the transaction header's actual positive timeout deadline to pass before invoking `sweepTimeouts()`. This keeps the test exercising the same timeout-sweeper path while removing the millisecond-boundary race. ### Verifying this change - [x] Make sure that the change passes the CI checks. ### Does this pull request potentially affect one of the following parts: - [ ] Dependencies (add or upgrade a dependency) - [ ] The public API - [ ] The schema - [ ] The default values of configurations - [ ] The threading model - [ ] The binary protocol - [ ] The REST endpoints - [ ] The admin CLI options - [ ] The metrics - [ ] Anything that affects deployment -- 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]
