Open
Description
The code block below in the selectOneMessageQueue method is identical to another overloaded method (2nd code block).
IMO, it can be rewritten like this.
for (int i = 0; i < this.messageQueueList.size(); i++) {
MessageQueue mq = selectOneMessageQueue()
if (!mq.getBrokerName().equals(lastBrokerName)) {
return mq;
}
}
If it's acceptable, I will submit a pull request.