apache / rocketmq

Apache RocketMQ is a cloud native messaging and streaming platform, making it simple to build event-driven applications.
https://rocketmq.apache.org/
Apache License 2.0
21.26k stars 11.7k forks source link

[Enhancement] Support JSON CQ Files and offset In-place Upgrade to RocksDB #8687

Closed LetLetMe closed 2 months ago

LetLetMe commented 2 months ago

Which Issue(s) This PR Fixes

Fixes #8589

之前的讨论参考:https://github.com/apache/rocketmq/pull/8600 Refer to the previous discussion at: https://github.com/apache/rocketmq/pull/8600

Brief Description

How Did You Test This Change?

codecov-commenter commented 2 months ago

Codecov Report

Attention: Patch coverage is 24.63235% with 205 lines in your changes missing coverage. Please review.

Project coverage is 47.49%. Comparing base (ba8148f) to head (8b1161f).

Files with missing lines Patch % Lines
...ocketmq/broker/processor/AdminBrokerProcessor.java 0.00% 61 Missing :warning:
...mand/queue/CheckRocksdbCqWriteProgressCommand.java 0.00% 38 Missing :warning:
...mq/broker/offset/RocksDBConsumerOffsetManager.java 52.94% 10 Missing and 6 partials :warning:
...org/apache/rocketmq/store/RocksDBMessageStore.java 0.00% 15 Missing :warning:
...org/apache/rocketmq/store/DefaultMessageStore.java 33.33% 11 Missing and 1 partial :warning:
...g/apache/rocketmq/client/impl/MQClientAPIImpl.java 0.00% 8 Missing :warning:
...ache/rocketmq/store/config/MessageStoreConfig.java 41.66% 7 Missing :warning:
.../subscription/RocksDBSubscriptionGroupManager.java 53.84% 4 Missing and 2 partials :warning:
...cketmq/broker/topic/RocksDBTopicConfigManager.java 50.00% 4 Missing and 2 partials :warning:
.../rocketmq/broker/offset/ConsumerOffsetManager.java 58.33% 3 Missing and 2 partials :warning:
... and 11 more
Additional details and impacted files ```diff @@ Coverage Diff @@ ## develop #8687 +/- ## ============================================= - Coverage 47.55% 47.49% -0.06% - Complexity 11546 11567 +21 ============================================= Files 1277 1280 +3 Lines 89449 89696 +247 Branches 11504 11540 +36 ============================================= + Hits 42535 42601 +66 - Misses 41706 41873 +167 - Partials 5208 5222 +14 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

LetLetMe commented 2 months ago

1111