[ISSUE #10651] Adapt RocketMQ to latest DLedger and remove fastjson1 - #10947
Draft
RongtongJin wants to merge 7 commits into
Draft
[ISSUE #10651] Adapt RocketMQ to latest DLedger and remove fastjson1#10947RongtongJin wants to merge 7 commits into
RongtongJin wants to merge 7 commits into
Conversation
added 7 commits
August 14, 2026 16:04
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Which Issue(s) This PR Fixes
Brief Description
This PR reworks the DLedger 0.3.3 adaptation on top of the latest DLedger code and openmessaging/dledger#336. It is intended to supersede the incomplete adaptation in #10650.
The change:
AppendFutureresult for a single-message batch andBatchAppendFuturefor multi-message batches;The validated compatibility boundary is a coordinated full-stop upgrade. Mixed-version rolling upgrade and downgrade are not covered by this PR.
Important
This is intentionally a draft. The current DLedger coordinate,
0.3.3-pr336-f2-64-SNAPSHOT, is a locally built artifact from DLedger master + PR #336 + fastjson2 2.0.64. Before this PR can be merged, DLedger #336 must be merged and a publicly resolvable DLedger artifact must be published; this PR must then switch to that coordinate and rerun the matrix.Related work: #10650, #10928, openmessaging/dledger#336.
How Did You Test This Change?
All Java verification used Amazon Corretto JDK 8.
clean test: 19/19 modules, 3,124 tests, 0 failures, 0 errors, 42 skipped.git diff --checkpassed.