ai-yang commented on PR #10697: URL: https://github.com/apache/rocketmq/pull/10697#issuecomment-5634069651
Refreshed onto `develop` at `1a50c6e4e` and pushed signed-off commit `e340eb4d4`. Resolved the import conflict in `DefaultMQProducerImplTest` by retaining both the upstream `EndTransactionRequestHeader` and regression-test `After` imports. The request-future cleanup implementation is unchanged by the rebase. Clean JDK 8 verification: `mvn -pl client -am -DskipITs -Dtest=DefaultMQProducerTest,DefaultMQProducerImplTest -Dsurefire.failIfNoSpecifiedTests=false clean test` passed 78/78 tests across the two classes. All 4 reactor modules passed with Checkstyle and SpotBugs enabled. The PR description now distinguishes this latest run from earlier full-reactor evidence. @drpmma @RongtongJin, could you take a human review of the synchronous-send-failure cleanup when convenient? The updated commit also triggers a new full CI run. -- 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]
