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 #9015] Sync SysFlag and message body inflation status; allow omit of message body#9016

Merged
lizhimins merged 1 commit intoapache:developapache/rocketmq:developfrom
lizhanhui:omit-message-bodylizhanhui/rocketmq:omit-message-bodyCopy head branch name to clipboard
Dec 5, 2024
Merged

[ISSUE #9015] Sync SysFlag and message body inflation status; allow omit of message body#9016
lizhimins merged 1 commit intoapache:developapache/rocketmq:developfrom
lizhanhui:omit-message-bodylizhanhui/rocketmq:omit-message-bodyCopy head branch name to clipboard

Conversation

@lizhanhui
Copy link
Copy Markdown
Contributor

Which Issue(s) This PR Fixes

Fixes #9015

Brief Description

How Did You Test This Change?

…ssage body

Signed-off-by: Li Zhanhui <lizhanhui@gmail.com>
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Dec 2, 2024

Codecov Report

Attention: Patch coverage is 46.15385% with 14 lines in your changes missing coverage. Please review.

Project coverage is 47.73%. Comparing base (9a891f1) to head (eb585fc).
Report is 159 commits behind head on develop.

Files with missing lines Patch % Lines
...e/rocketmq/client/producer/ProduceAccumulator.java 50.00% 0 Missing and 6 partials ⚠️
...he/rocketmq/client/impl/consumer/ProcessQueue.java 28.57% 0 Missing and 5 partials ⚠️
...ent/trace/hook/SendMessageOpenTracingHookImpl.java 66.66% 0 Missing and 1 partial ⚠️
...mq/client/trace/hook/SendMessageTraceHookImpl.java 50.00% 0 Missing and 1 partial ⚠️
...apache/rocketmq/common/message/MessageDecoder.java 50.00% 1 Missing ⚠️
Additional details and impacted files
@@              Coverage Diff              @@
##             develop    #9016      +/-   ##
=============================================
- Coverage      47.74%   47.73%   -0.02%     
- Complexity     11802    11806       +4     
=============================================
  Files           1305     1305              
  Lines          91372    91385      +13     
  Branches       11733    11746      +13     
=============================================
- Hits           43626    43621       -5     
- Misses         42360    42362       +2     
- Partials        5386     5402      +16     

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

@lizhimins lizhimins merged commit 01a5123 into apache:develop Dec 5, 2024
RongtongJin pushed a commit that referenced this pull request Jul 19, 2025
ChineseTony pushed a commit to ChineseTony/rocketmq that referenced this pull request Aug 8, 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] Omit message body causes NPE

4 participants

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