humkum opened a new issue, #8168:
URL: https://github.com/apache/rocketmq/issues/8168
### Before Creating the Bug Report
- [X] I found a bug, not just asking a question, which should be created in
[GitHub Discussions](https://github.com/apache/rocketmq/discussions).
- [X] I have searched the [GitHub
Issues](https://github.com/apache/rocketmq/issues) and [GitHub
Discussions](https://github.com/apache/rocketmq/discussions) of this
repository and believe that this is not a duplicate.
- [X] I have confirmed that this bug belongs to the current repository, not
other repositories of RocketMQ.
### Runtime platform environment
CentOS7.3
### RocketMQ version
5.2.0
### JDK Version
Open_JDK 1.8.0_202
### Describe the Bug
When producer send a message in async mode, if the request is timeout with
throwing a RemotingTooMuchException, the producer would retry with a negative
value with timeoutMillis, which is not necessary.
![image](https://github.com/apache/rocketmq/assets/50660789/9e4e6c02-839c-48b8-92b1-068ba5032183)
![image](https://github.com/apache/rocketmq/assets/50660789/f7a58a1c-e20f-47bd-9a06-3a6b24784474)
### Steps to Reproduce
Producer add a RPCHook
![image](https://github.com/apache/rocketmq/assets/50660789/45face5d-784a-40a8-9efc-1151538fde08)
Try sleep 4s in in the doBeforeRequest function,to make the request cost
time out.
### What Did You Expect to See?
No retry when the request is already timeout.
### What Did You See Instead?
The producer retry with a negative value of timeout, which will must failed,
and it's unnecessary.
### Additional Context
_No response_
--
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: commits-unsubscr...@rocketmq.apache.org.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org