wuguanyu commented on issue #3477:
URL: https://github.com/apache/rocketmq/issues/3477#issuecomment-965112349


   I see source code carefully 
   
   ` if (findBrokerResult != null) {
               {
                   // check version
                   if (!ExpressionType.isTagType(expressionType)
                       && findBrokerResult.getBrokerVersion() < 
MQVersion.Version.V4_1_0_SNAPSHOT.ordinal()) {
                       throw new MQClientException("The broker[" + 
mq.getBrokerName() + ", "
                           + findBrokerResult.getBrokerVersion() + "] does not 
upgrade to support for filter message by " + expressionType, null);
                   }
               }`
   
   findBrokerResult.getBrokerVersion() 
   this brokerVersion from local variable  brokerVersionTable 
   brokerVersionTable will be init in sendHeartbeatToAllBroker 
   
   I guess  findBrokerResult.getBrokerVersion() should be 
MQVersion.Version.V4_7_1_SNAPSHOT.ordinal()
   
   but brokerVersion result is 0


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to