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/01/26 10:24:31 UTC

[GitHub] [rocketmq] d4ksn opened a new pull request #2632: [ISSUE #2621] Fix ProcessQueue not locked after consumer rebalance

d4ksn opened a new pull request #2632:
URL: https://github.com/apache/rocketmq/pull/2632


   ## What is the purpose of the change
   
   For-#2621
   
   ## Brief changelog
   
   If an orderly consumer gets assigned a new queue, and gets locked in the broker successfully, set the newly created ProcessQueue to locked state.
   
   ## Verifying this change
   
   After this change, there should be almost no consumer delay when an orderly consumer starts consuming messages
   


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

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



[GitHub] [rocketmq] duhenglucky commented on pull request #2632: [ISSUE #2621] Fix ProcessQueue not locked after consumer rebalance

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


   @d4ksn May require you to re-submit the PR, because the CI build failed and the record was not found 


-- 
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 #2632: [ISSUE #2621] Fix ProcessQueue not locked after consumer rebalance

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


   
   [![Coverage Status](https://coveralls.io/builds/36692635/badge)](https://coveralls.io/builds/36692635)
   
   Coverage increased (+0.08%) to 51.745% when pulling **2c6ada6f452caa8ff24ed9a181d8cfaab72d1d85 on d4ksn:develop** into **d8f88c51eb50e43a6bce3fce1ed14f49714ff108 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.

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



[GitHub] [rocketmq] coveralls edited a comment on pull request #2632: [ISSUE #2621] Fix ProcessQueue not locked after consumer rebalance

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


   
   [![Coverage Status](https://coveralls.io/builds/41111218/badge)](https://coveralls.io/builds/41111218)
   
   Coverage increased (+0.03%) to 54.039% when pulling **738ca07ef48113a72ebca406356825734d615b7e on d4ksn:develop** into **2a6371765e9382b4d6d4e9daf1b65c0ebaa4cf96 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] coveralls edited a comment on pull request #2632: [ISSUE #2621] Fix ProcessQueue not locked after consumer rebalance

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


   
   [![Coverage Status](https://coveralls.io/builds/41111218/badge)](https://coveralls.io/builds/41111218)
   
   Coverage increased (+0.03%) to 54.039% when pulling **738ca07ef48113a72ebca406356825734d615b7e on d4ksn:develop** into **2a6371765e9382b4d6d4e9daf1b65c0ebaa4cf96 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] d4ksn commented on pull request #2632: [ISSUE #2621] Fix ProcessQueue not locked after consumer rebalance

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


   @RongtongJin 
   Hello! I have fix the conflict, can it be merged to master?


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