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

[ISUUE #8762] Fix dLedger lock granularity issue causing LMQ message loss #8763

Closed bxfjb closed 4 days ago

bxfjb commented 1 month ago

Which Issue(s) This PR Fixes

Fixes #8762

Brief Description

How Did You Test This Change?

codecov-commenter commented 1 month ago

Codecov Report

Attention: Patch coverage is 12.50000% with 35 lines in your changes missing coverage. Please review.

Project coverage is 47.30%. Comparing base (daf3d1a) to head (dd99891).

Files with missing lines Patch % Lines
...pache/rocketmq/store/queue/MultiDispatchUtils.java 3.57% 27 Missing :warning:
...pache/rocketmq/store/dledger/DLedgerCommitLog.java 33.33% 6 Missing and 2 partials :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## develop #8763 +/- ## ============================================= - Coverage 47.51% 47.30% -0.21% + Complexity 11589 11548 -41 ============================================= Files 1282 1282 Lines 89848 89888 +40 Branches 11557 11568 +11 ============================================= - Hits 42689 42522 -167 - Misses 41932 42118 +186 - Partials 5227 5248 +21 ``` | [Flag](https://app.codecov.io/gh/apache/rocketmq/pull/8763/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Coverage Δ | | |---|---|---| | [](https://app.codecov.io/gh/apache/rocketmq/pull/8763/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `47.30% <12.50%> (-0.21%)` | :arrow_down: | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#carryforward-flags-in-the-pull-request-comment) to find out more.

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