Skip to content

[ISSUE #9235] fix: response codes do not match#9236

Closed
3424672656 wants to merge 3 commits intoapache:developfrom
3424672656:fix_mismatch
Closed

[ISSUE #9235] fix: response codes do not match#9236
3424672656 wants to merge 3 commits intoapache:developfrom
3424672656:fix_mismatch

Conversation

@3424672656
Copy link
Copy Markdown
Contributor

Which Issue(s) This PR Fixes

Fixes #9235

Brief Description

How Did You Test This Change?

@codecov-commenter
Copy link
Copy Markdown

Codecov Report

Attention: Patch coverage is 0% with 2 lines in your changes missing coverage. Please review.

Project coverage is 48.02%. Comparing base (535a078) to head (03239c7).
Report is 3 commits behind head on develop.

Files with missing lines Patch % Lines
...ocketmq/broker/processor/AdminBrokerProcessor.java 0.00% 2 Missing ⚠️
Additional details and impacted files
@@              Coverage Diff              @@
##             develop    #9236      +/-   ##
=============================================
- Coverage      48.02%   48.02%   -0.01%     
+ Complexity     12095    12094       -1     
=============================================
  Files           1322     1322              
  Lines          93094    93103       +9     
  Branches       11933    11935       +2     
=============================================
+ Hits           44709    44710       +1     
- Misses         42847    42856       +9     
+ Partials        5538     5537       -1     

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


response.setCode(ResponseCode.CONSUMER_NOT_ONLINE);
response.setRemark("the consumer group[" + requestHeader.getConsumerGroup() + "] not online");
response.setCode(ResponseCode.SUBSCRIPTION_GROUP_NOT_EXIST);
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

这种情况下有可能是CONSUMER_NOT_ONLINE,也有可能是SUBSCRIPTION_GROUP_NOT_EXIST

@3424672656 3424672656 requested a review from RongtongJin March 12, 2025 09:04
@3424672656 3424672656 closed this Mar 14, 2025
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.

[Bug] Response codes do not match

3 participants