unbridled-41 opened a new pull request, #5971:
URL: https://github.com/apache/rocketmq-dashboard/pull/5971

   ### Which Issue(s) This PR Fixes
   Fixes #5970
   ### Problem / Evidence
   `deploy/rocketmq/README.md` lines 20-21 said both generators connect via 
`proxy:8080`, while `clients/TraceProducer.java:7` / `TraceConsumer.java:8` 
read `NAMESRV_ADDR` and `docker-compose.yml` passes `rmq-nameserver:9876` to 
both. Its own comments say 直连 NameServer, the runbook agrees, and no client 
reads a proxy or gRPC endpoint.
   ### Root cause / Fix
   Stale directory comment from an earlier proxy-mode plan; correct it to name 
the NameServer connection.
   ### Priority and scoring
   **PRIORITY 45** - impact 12/40 (misleading orientation, no code impact), 
blast radius 8/20 (the demo stack), reproducibility 20/20 (grep), maintenance 
value 5/20. **FIX_CONFIDENCE 90**.
   ### Tests
   Documentation only; verified with `grep -n "NAMESRV_ADDR\|setNamesrvAddr" 
deploy/rocketmq/clients/*.java deploy/rocketmq/docker-compose.yml` (every 
client/container uses the NameServer) and `grep -rn "proxy\|8080" 
deploy/rocketmq/clients/` (no hits).
   ### Risk
   None.
   


-- 
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