ShadowySpirits commented on issue #1576: Bug in select MessageQueue URL: https://github.com/apache/rocketmq/issues/1576#issuecomment-557357898 > > @duhenglucky In my opinion, lastBrokerName represents the broker that should be avoided. But in the current logic is the broker that should be selected. > > In my opinion, lastBrokerName(means last selected broker) whether should be isolated only depends on isAvailable() method, instead of adding judgment logic elsewhere, if the broker is not isolated, messages should be sent to the broker directly. This will also make the load of each broker more balanced. According to your explanation, I submitted pr #1607 to delete the if statement.
---------------------------------------------------------------- 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: [email protected] With regards, Apache Git Services
