QianZhenyu created ROCKETMQ-37:
----------------------------------

             Summary: Log output information is not accurate
                 Key: ROCKETMQ-37
                 URL: https://issues.apache.org/jira/browse/ROCKETMQ-37
             Project: Apache RocketMQ
          Issue Type: Improvement
            Reporter: QianZhenyu
            Assignee: vongosling


1. in MQClientManager#getAndCreateMQClientInstance  line 57
when generate a new key-value pair for client  in factoryTable,   if there was 
no mapping for the clientId previously, putIfAbsent will return null, and then 
output:
"log.warn("Previous MQClientInstance has created for clientId:[{}]", clientId);"
this is wrong, it should output in condition prev != null

2. in SubscriptionGroupManager#deleteSubscriptionGroupConfig  line 187
if program execute:
log.warn("delete subscription group failed, subscription group: {} not exist", 
old);
variable old is always null , it's useless, so it's better to  print groupName




--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to