You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@rocketmq.apache.org by GitBox <gi...@apache.org> on 2022/11/21 04:37:41 UTC

[GitHub] [rocketmq] drpmma commented on issue #5555: create producer instance name with PID + nano by default is seems not a good idea

drpmma commented on issue #5555:
URL: https://github.com/apache/rocketmq/issues/5555#issuecomment-1321458143

   It's a matter of the default behavior whether to share MQClientInstance. It was indeed designed for sharing MQClientInstance when RocketMQ was implemented first. However, MQClientInstance's sharing may cause issues like unexpected connection close and the repeated clientId which will lead to wrong rebalance result.
   
   So I think it's better to keep it for not sharing by default and user can set the same instaceName when sharing is need.
   


-- 
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: commits-unsubscribe@rocketmq.apache.org

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