MQFaultStrategy#选择一条消息队列问题
public MessageQueue selectOneMessageQueue(final TopicPublishInfo tpInfo, final String lastBrokerName) { if (this.sendLatencyFaultEnable) { try { int index = tpInfo.getSendWhichQueue().getAndIncrement(); for (int i = 0; i < tpInfo.getMessageQueueList().size(); i++) { int pos = Math.abs(index++) % tpInfo.getMessageQueueList().size(); if (pos < 0) pos = 0; MessageQueue mq = tpInfo.getMessageQueueList().get(pos); if (latencyFaultTolerance.isAvailable(mq.getBrokerName())) { if (null == lastBrokerName || mq.getBrokerName().equals(lastBrokerName)) return mq; } } ...
if (null == lastBrokerName || mq.getBrokerName().equals(lastBrokerName)) return mq;
应该是：
if (null == lastBrokerName || !mq.getBrokerName().equals(lastBrokerName)) return mq;
如果Broker可用，则确定它是否是第一选择（null==lastBrokerName），如果是第一选择，则返回mq，如果不是，则应使用新mq，但不应使用lastBrokerName（！mq.getBrokerName（）.equals（lastBroker name）），因为lastBroker名称是fail Broker名称。所以我认为当前的代码是不正确的。
参考代码（TopicPublishInfo#selectOneMessageQueue）：
原提问者GitHub用户mushanshitiancai
写回答
-
这种策略允许生产者对可用的代理进行优先级排序。RocketMQ还提供了配置（sendLatencyFaultEnable）来对其进行更多操作。
原回答者GitHub用户duhenglucky
2023-05-26 17:57:29赞同 展开评论 打赏
版权声明：本文内容由阿里云实名注册用户自发贡献，版权归原作者所有，阿里云开发者社区不拥有其著作权，亦不承担相应法律责任。具体规则请查看《阿里云开发者社区用户服务协议》和《阿里云开发者社区知识产权保护指引》。如果您发现本社区中有涉嫌抄袭的内容，填写侵权投诉表单进行举报，一经查实，本社区将立刻删除涉嫌侵权内容。
阿里云拥有国内全面的云原生产品技术以及大规模的云原生应用实践，通过全面容器化、核心技术互联网化、应用 Serverless 化三大范式，助力制造业企业高效上云，实现系统稳定、应用敏捷智能。拥抱云原生，让创新无处不在。
