[I] [Enhancement] Upgrade DLedger to 0.3.3-SNAPSHOT [rocketmq]

["leizhiyuan (via GitHub)" <gi...@apache.org>]

leizhiyuan opened a new issue, #10651:
URL: https://github.com/apache/rocketmq/issues/10651

   ### Before Creating the Enhancement Request
   
   - [x] I have confirmed that this should be classified as an enhancement rather than a bug/feature.
   
   ### Summary
   
   Upgrade RocketMQ's DLedger dependency to `0.3.3-SNAPSHOT`.
   
   ### Motivation
   
   RocketMQ currently depends on DLedger `0.3.2`. Upgrading to `0.3.3-SNAPSHOT` helps RocketMQ keep aligned with the latest DLedger changes and removes the need to keep the corresponding `fastjson1` dependency in RocketMQ, because the newer DLedger version no longer depends on `fastjson1`.
   
   ### Describe the Solution You'd Like
   
   Upgrade the DLedger version from `0.3.2` to `0.3.3-SNAPSHOT` and adapt RocketMQ to the related DLedger API changes, including:
   
   - Update moved DLedger future classes to the new package.
   - Replace deprecated or removed DLedger configuration and store APIs.
   - Adapt DLedger controller state machine code to the new apply iterator API.
   - Remove the obsolete `fastjson1` dependency and use `fastjson2` where necessary.
   
   ### Describe Alternatives You've Considered
   
   Keep using DLedger `0.3.2` and retain the existing dependency setup. However, this leaves RocketMQ behind the latest DLedger updates and keeps the no-longer-needed `fastjson1` dependency in the project.
   
   ### Additional Context
   
   A corresponding pull request will be linked to this issue.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@rocketmq.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [I] [Enhancement] Upgrade DLedger to 0.3.3-SNAPSHOT [rocketmq]

["RockteMQ-AI (via GitHub)" <gi...@apache.org>]

RockteMQ-AI commented on issue #10651:
URL: https://github.com/apache/rocketmq/issues/10651#issuecomment-5054198245

   **Issue Evaluation**
   
   Category: `type/enhancement` | Status: **Evaluated**
   
   **Feasibility:** feasible
   **Scope:** DLedger integration layer — `dledger` module, controller state machine, `pom.xml` dependency management
   **Compatibility:** Low risk — DLedger 0.3.3 is a minor version bump; API adaptations are clearly scoped (future class package moves, config/store API changes, controller apply iterator API). Removing `fastjson1` in favor of `fastjson2` aligns with the broader project direction.
   
   This is a well-defined dependency upgrade with clear motivation (fastjson1 removal + DLedger alignment). The scope and adaptation points are well documented.
   
   If you already have a PR in progress, feel free to link it. Otherwise, an automated fix proposal can be generated — reply to proceed.
   
   ---
   *Automated evaluation by RockteMQ-AI*


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org