chennaji9 opened a new pull request, #11331:
URL: https://github.com/apache/rocketmq/pull/11331

   <!-- Please make sure the target branch is right. In most case, the target 
branch should be `develop`. -->
   
   ### Which Issue(s) This PR Fixes
   
   Fixes #11323
   
   ### Brief Description
   
   The request-reply demo `ResponseConsumer` printed the exception and still 
returned `CONSUME_SUCCESS` when sending the reply failed, so the request was 
acknowledged as consumed while the requester never received a reply (its 
`request()` would just time out) — even though a redelivery would normally 
produce the reply. The listener now returns `RECONSUME_LATER` on 
`MQClientException`/`RemotingException`/`MQBrokerException` so the request is 
redelivered and the reply retried, and `InterruptedException` restores the 
interrupt flag before returning. The identical snippet embedded in 
`docs/cn/rpc_request.md` is updated to match.
   
   ### How Did You Test This Change?
   
   `mvn -pl example -am -DskipTests compile` (JDK 11, PR base commit) → BUILD 
SUCCESS with the change. Behavior reasoning: `RECONSUME_LATER` triggers the 
standard push-consumer redelivery with delay (retry topic → max retries → DLQ), 
which is the documented pattern used by the simple consumer examples for 
transient send failures; an example-module test harness does not exist, so no 
new unit test was added.
   
   🤖 Generated with [Claude Code](https://claude.com/claude-code)


-- 
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]

Reply via email to