ai-yang commented on PR #10986: URL: https://github.com/apache/rocketmq/pull/10986#issuecomment-5651391792
Pushed test-only follow-up a191a2953750e946c2ffc6d8ba1d9cb547e85036; the production fix is unchanged. The exceptional-read regression now passes through a real EscapeBridge and injects failure at MessageStore.getMessageAsync. A second case holds the read pending until after revive offset 1 is committed, then completes it exceptionally. It verifies exactly one replacement CK submission and removal of the in-flight entry. Latest local JDK 8 / Maven validation: PopReviveServiceTest 14/14 passed; all 11 targeted reactor modules succeeded with Checkstyle and SpotBugs enabled. The PR description now distinguishes store-future fault injection from physical storage failures/durable persistence, and the historical baseline reproduction from today's expanded test run. This additional coverage is available for the existing human review and coordination with #11023. The old head's green CI does not certify this new head; its checks should be considered separately. -- 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]
