You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@rocketmq.apache.org by GitBox <gi...@apache.org> on 2021/07/27 10:05:49 UTC

[GitHub] [rocketmq] lwclover opened a new pull request #3182: registerProducer & unregisterProducer add synchronization for producer

lwclover opened a new pull request #3182:
URL: https://github.com/apache/rocketmq/pull/3182


   在pr [3120](https://github.com/apache/rocketmq/pull/3120)中,对consumer启动、停止过程加锁防止出现幽灵consumer的问题。
   其中同学Zanglei06建议我把producer也加上锁,考虑到虽然产生幽灵producer对业务无影响,但是为了严谨性提交了此pr


-- 
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



[GitHub] [rocketmq] coveralls commented on pull request #3182: add synchronization for producer on method registerProducer & unregisterProducer

Posted by GitBox <gi...@apache.org>.
coveralls commented on pull request #3182:
URL: https://github.com/apache/rocketmq/pull/3182#issuecomment-887664691


   
   [![Coverage Status](https://coveralls.io/builds/41678272/badge)](https://coveralls.io/builds/41678272)
   
   Coverage increased (+0.05%) to 54.087% when pulling **72128f76ab1b4338c2841f0d2ec1794a26448d92 on lwclover:develop** into **36e6687d50b2de14d878dbc68e1b99fe254fed67 on apache:develop**.
   


-- 
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



[GitHub] [rocketmq] lwclover commented on pull request #3182: registerProducer & unregisterProducer add synchronization for producer

Posted by GitBox <gi...@apache.org>.
lwclover commented on pull request #3182:
URL: https://github.com/apache/rocketmq/pull/3182#issuecomment-887384309


   @duhenglucky  help me on it.


-- 
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



[GitHub] [rocketmq] lwclover edited a comment on pull request #3182: add synchronization for producer on method registerProducer & unregisterProducer

Posted by GitBox <gi...@apache.org>.
lwclover edited a comment on pull request #3182:
URL: https://github.com/apache/rocketmq/pull/3182#issuecomment-887384309


   @duhenglucky  @RongtongJin  please help me on it.


-- 
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



[GitHub] [rocketmq] duhenglucky merged pull request #3182: [ISSUE #3183]add synchronization for producer on method registerProducer & unregisterProducer

Posted by GitBox <gi...@apache.org>.
duhenglucky merged pull request #3182:
URL: https://github.com/apache/rocketmq/pull/3182


   


-- 
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