[GitHub] [rocketmq] wuguanyu edited a comment on issue #3477: pullKernelImpl exception org.apache.rocketmq.client.exception.MQClientException

[GitBox <gi...@apache.org>]

wuguanyu edited a comment 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()
    rocketmq.remoting.version          355
   
   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: dev-unsubscribe@rocketmq.apache.org

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