You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@rocketmq.apache.org by GitBox <gi...@apache.org> on 2020/10/16 11:29:52 UTC

[GitHub] [rocketmq] q2wxec opened a new issue #2358: It seems that when sendLatencyFaultEnable is on，clinet force you to choose the fail lastBroker

q2wxec opened a new issue #2358:
URL: https://github.com/apache/rocketmq/issues/2358


   
   **BUG REPORT**
   
   1. Please describe the issue you observed:
   IN  org.apache.rocketmq.client.latency.MQFaultStrategy#selectOneMessageQueue
   
   public MessageQueue selectOneMessageQueue(final TopicPublishInfo tpInfo, final String lastBrokerName) {
           if (this.sendLatencyFaultEnable) {
              
                 .....
                       if (latencyFaultTolerance.isAvailable(mq.getBrokerName())) {
                           // I think the condition here should be 
                           //if (null == lastBrokerName || !mq.getBrokerName().equals(lastBrokerName))
                           //otherwise clinet force you to choose the fail lastBroker
                           if (null == lastBrokerName || mq.getBrokerName().equals(lastBrokerName))
                               return mq;
                       }
                   }
   
   
   2. Please tell us about your environment:
   client: 4.7.0
   
   3. Other information (e.g. detailed explanation, logs, related issues, suggestions how to fix, etc):
   
   
   


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

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [rocketmq] q2wxec closed issue #2358: It seems that when sendLatencyFaultEnable is on，clinet force you to choose the fail lastBroker

Posted by GitBox <gi...@apache.org>.
q2wxec closed issue #2358:
URL: https://github.com/apache/rocketmq/issues/2358


   


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

For queries about this service, please contact Infrastructure at:
users@infra.apache.org