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

   **Issue Evaluation**
   
   Category: `bug` | Status: **Not Reproducible**
   
   After verifying against the current codebase, the specific claims in this 
issue do not match the actual code:
   
   | Claim | Finding |
   |-------|---------|
   | "BrokerVO.version declared as required" | No `BrokerVO` class/interface 
exists. Broker data is a raw `Map<String,String>` on the backend and an untyped 
spread object on the frontend. |
   | "cluster broker table calls `localeCompare` when sorting" | `cluster.jsx` 
has zero `localeCompare` calls and no `sorter` props on any column. The only 
`localeCompare` in the codebase is in `consumer.jsx:122`, unrelated to the 
cluster view. |
   | "runtime-unavailable Broker causes a client-side error" | The VERSION 
column (`cluster.jsx:138`) uses `dataIndex: 'brokerVersionDesc'` with no render 
fallback — Ant Design renders the cell empty silently. No JS crash. |
   
   **Related finding:** `ClusterServiceImpl.list()` has no per-broker exception 
handling — if any single broker is unreachable, `fetchBrokerRuntimeStats` 
throws and the entire API call fails with a 500. This is a real server-side 
issue but different from what is described here.
   
   This issue does not appear to be a valid bug as described. If you can 
provide a specific reproduction (browser console error, stack trace), we can 
re-evaluate.
   
   ---
   *Automated evaluation by github-manager-bot*


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