Skip to content

Conversation

@f1amingo
Copy link
Contributor

@f1amingo f1amingo commented Nov 3, 2025

Which Issue(s) This PR Fixes

Fixes #9797

Brief Description

RIP‐83 Lite Topic: A New Message Model
https://github.com/apache/rocketmq/wiki/RIP%E2%80%9083-Lite-Topic:-A-New-Message-Model

How Did You Test This Change?

@hill007299 hill007299 requested review from RongtongJin, fuyou001, lizhimins and lollipopjin and removed request for lizhimins November 5, 2025 03:28
@RongtongJin
Copy link
Contributor

CI need to be passed
image

@lizhimins lizhimins added this to the 5.4.0 milestone Nov 10, 2025
@codecov-commenter
Copy link

codecov-commenter commented Nov 12, 2025

Codecov Report

❌ Patch coverage is 41.60000% with 1898 lines in your changes missing coverage. Please review.
✅ Project coverage is 48.64%. Comparing base (beee84f) to head (2c1940d).
⚠️ Report is 1 commits behind head on develop.

Files with missing lines Patch % Lines
...etmq/broker/lite/LiteSubscriptionRegistryImpl.java 14.97% 159 Missing ⚠️
...etmq/broker/processor/PopLiteMessageProcessor.java 53.30% 97 Missing and 16 partials ⚠️
.../broker/longpolling/PopLiteLongPollingService.java 41.89% 74 Missing and 12 partials ⚠️
...tools/command/lite/GetLiteGroupInfoSubCommand.java 0.00% 86 Missing ⚠️
...g/apache/rocketmq/client/impl/MQClientAPIImpl.java 1.19% 83 Missing ⚠️
...rocketmq/broker/processor/AckMessageProcessor.java 3.33% 55 Missing and 3 partials ⚠️
...ache/rocketmq/broker/lite/LiteEventDispatcher.java 78.92% 41 Missing and 14 partials ⚠️
...ools/command/lite/GetBrokerLiteInfoSubCommand.java 0.00% 53 Missing ⚠️
...ools/command/lite/GetLiteClientInfoSubCommand.java 0.00% 53 Missing ⚠️
...broker/processor/LiteSubscriptionCtlProcessor.java 8.77% 52 Missing ⚠️
... and 96 more
Additional details and impacted files
@@              Coverage Diff              @@
##             develop    #9800      +/-   ##
=============================================
- Coverage      48.91%   48.64%   -0.27%     
- Complexity     12770    13188     +418     
=============================================
  Files           1324     1370      +46     
  Lines          96288    99431    +3143     
  Branches       12409    12835     +426     
=============================================
+ Hits           47097    48366    +1269     
- Misses         43458    45189    +1731     
- Partials        5733     5876     +143     

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

@f1amingo f1amingo force-pushed the feature/lite_topic branch 7 times, most recently from 2d9967b to 5e8d18a Compare November 13, 2025 12:56
@f1amingo f1amingo force-pushed the feature/lite_topic branch 2 times, most recently from 3724400 to 065b58e Compare December 12, 2025 09:46
f1amingo and others added 2 commits December 15, 2025 14:41
Change-Id: Ie83f5ad363eb7b7c48371a0f4b3eab9f04465b81
Change-Id: I102d2bf7b5d573f41416333135074cde1a9f1fec
Change-Id: Iadc1e2c74f1a05954c4015faa92cefb5786cdc0f
Change-Id: I360bfb5d2ee12d2f5cc7a72343872c12c1bc5bd5
@f1amingo f1amingo force-pushed the feature/lite_topic branch 2 times, most recently from fbef7e3 to 8688da8 Compare December 16, 2025 02:49
Change-Id: If8174ba1e35c899e8fa9067923fea7ecb69291e1

public static final int POP_LITE_MESSAGE = 200070;
public static final int LITE_SUBSCRIPTION_CTL = 200071;
public static final int ACK_LITE_MESSAGE = 200072;
Copy link
Member

Choose a reason for hiding this comment

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

ACK_LITE_MESSAGE is not used

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.

[Feature] RIP-83 Lite Topic: A New Message Model

6 participants