Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

[ISSUE #7547] Let consumer be aware of message queue assignment change#7548

Merged
lizhanhui merged 7 commits intodevelopapache/rocketmq:developfrom
consumer_aware_queue_changeapache/rocketmq:consumer_aware_queue_changeCopy head branch name to clipboard
Nov 13, 2023
Merged

[ISSUE #7547] Let consumer be aware of message queue assignment change#7548
lizhanhui merged 7 commits intodevelopapache/rocketmq:developfrom
consumer_aware_queue_changeapache/rocketmq:consumer_aware_queue_changeCopy head branch name to clipboard

Conversation

@lizhanhui
Copy link
Copy Markdown
Contributor

fix #7547

Signed-off-by: Li Zhanhui <lizhanhui@gmail.com>
Signed-off-by: Li Zhanhui <lizhanhui@gmail.com>
@lizhanhui lizhanhui changed the title let consumer be aware of message queue assignment change [ISSUE #7547] Let consumer be aware of message queue assignment change Nov 10, 2023
lizhanhui and others added 4 commits November 10, 2023 15:49
Signed-off-by: Zhanhui Li <lizhanhui@apache.org>
Signed-off-by: Zhanhui Li <lizhanhui@apache.org>
Signed-off-by: Li Zhanhui <lizhanhui@gmail.com>
Signed-off-by: Li Zhanhui <lizhanhui@gmail.com>
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Nov 10, 2023

Codecov Report

❌ Patch coverage is 45.45455% with 6 lines in your changes missing coverage. Please review.
✅ Project coverage is 43.08%. Comparing base (27759f3) to head (4552c33).
⚠️ Report is 647 commits behind head on develop.

Files with missing lines Patch % Lines
...ocketmq/client/consumer/DefaultMQPushConsumer.java 33.33% 2 Missing ⚠️
...lient/impl/consumer/DefaultMQPushConsumerImpl.java 50.00% 1 Missing and 1 partial ⚠️
...cketmq/client/impl/consumer/RebalancePushImpl.java 33.33% 1 Missing and 1 partial ⚠️
Additional details and impacted files
@@              Coverage Diff              @@
##             develop    #7548      +/-   ##
=============================================
- Coverage      43.13%   43.08%   -0.06%     
+ Complexity      9709     9708       -1     
=============================================
  Files           1159     1159              
  Lines          84049    84059      +10     
  Branches       10923    10924       +1     
=============================================
- Hits           36257    36216      -41     
- Misses         43279    43326      +47     
- Partials        4513     4517       +4     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Signed-off-by: Zhanhui Li <lizhanhui@apache.org>
@lizhanhui lizhanhui requested review from aaron-ai, drpmma and xdkxlk and removed request for drpmma November 10, 2023 11:45
@lizhanhui lizhanhui merged commit 15d32db into develop Nov 13, 2023
@lizhanhui lizhanhui deleted the consumer_aware_queue_change branch November 13, 2023 01:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Enhancement] Let PushConsumer client aware of message queue assignment change

3 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.