RockteMQ-AI commented on issue #1057:
URL: 
https://github.com/apache/rocketmq-dashboard/issues/1057#issuecomment-5199550798

   **Issue Evaluation**
   
   Category: `enhancement` | Status: **Evaluated**
   
   **Feasibility:** Feasible
   **Scope:** Runtime operations layer (AdminClient resolution), DLQ 
list/resend as first consumer
   **Compatibility:** No breaking changes — introduces instance-scoped 
resolution alongside existing singleton path
   
   Well-structured proposal. Replacing the process-wide `DefaultMQAdminExt` 
singleton with instance-scoped resolution is the correct architectural 
direction for multi-instance support. Using `MqAdminExtFactory` cache for 
endpoint-to-client mapping avoids redundant connection overhead. Establishing 
DLQ list/resend as the first consumer provides a focused validation slice 
before broader migration.
   
   The foundation-resolver approach (resolve `instanceId` → endpoint → cached 
AdminClient) is clean and reusable for the planned follow-ups (client 
connections, message query, trace).
   
   ---
   *Automated evaluation by github-manager*


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to