lizhimins opened a new pull request, #10378: URL: https://github.com/apache/rocketmq/pull/10378
## Summary - Fix root causes of 4 flaky tests quarantined in #10374 and remove `@Ignore` annotations - Remove `rerun-workflow.yml` to stop masking flaky tests with automatic CI retries ## Changes | Test | Root Cause | Fix | |------|-----------|-----| | `BloomFilterTest#testCheckFalseHit` | `new Random(System.nanoTime())` per-character produced duplicate strings | Use single seeded `Random(42)` instance | | `TransactionalMessageServiceImplTest#testDeletePrepareMessage_maxSize` | `verify(bridge, timeout(50))` too short for slow CI | Increase to `timeout(3000)` | | `DefaultMQConsumerWithTraceTest#testPullMessage_WithTrace_Success` | Async `PullMessageService` race condition | Call `pullMessage` directly | | `DefaultMQLitePullConsumerWithTraceTest` (2 methods) | Instance-level `RebalanceService.waitInterval` race | Set as static field in `@Before` | ## Test plan - [x] BloomFilterTest: 478/1000 runs passed (zero failures) before manual stop - [x] TransactionalMessageServiceImplTest: 1000/1000 passed - [x] DefaultMQConsumerWithTraceTest: 5000x verification in progress - [x] DefaultMQLitePullConsumerWithTraceTest: ~900/1000 passed before manual stop -- 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]
