zjncs opened a new issue, #6656:
URL: https://github.com/apache/rocketmq-dashboard/issues/6656

   ## Problem
   
   After #950 resolved #769 by replacing the fabricated `Math.random()` runtime 
values with `null`, the Proxy page kept rendering the columns and stat cards 
those fields fed:
   
   - The node table still renders six runtime columns — 版本 (version), 连接数 
(connections), TPS, 内存 (memory), CPU, 运行时间 (uptime) — that no endpoint fills. 
`ProxyHomePageData` carries only the address list and the current address; 
`ProxyTopologyVO` carries status, ports and latency. Every row permanently 
shows the unavailable placeholder, which reads as "metrics temporarily missing" 
rather than "metrics do not exist".
   - The stats row still renders 总连接数 (total connections) and 总 TPS (total TPS) 
cards, permanently `N/A`.
   - 健康节点 (healthy nodes) is hardcoded `null` even though the page already 
overlays the live topology status onto every node — the count is never computed.
   
   #769's expected behavior allowed two outcomes: "hide unsupported runtime 
columns or show an explicit unavailable state such as N/A". #950 chose the N/A 
route; the permanently-empty columns and cards are the residue of that choice.
   
   ## Expected behavior
   
   - Drop the columns and aggregate cards no endpoint can fill (same direction 
as the merged #6145, which removed a panel the API never fills).
   - Compute the healthy node count from the overlaid topology status, so the 
健康节点 card answers instead of showing `N/A` forever.
   
   ## Evidence
   
   Reproduced with the frontend tests (`cd web && npx vitest run 
src/pages/studio/__tests__/Proxy.test.tsx`):
   
   - `getProxyTopology` resolving 
`[{status:'UP'},{status:'UP'},{status:'DOWN'}]` leaves the 健康节点 card rendering 
`N/A`;
   - the six runtime column headers and the two aggregate cards render with 
placeholder content in every state.
   
   `web/src/pages/studio/Proxy.tsx` `applyProxyHome` initializes 
`version/connections/tps/memory/cpu/uptime` to `null` and calls 
`setClusterStats({..., healthyNodes: null, totalConnections: null, totalTPS: 
null})`; nothing else in the page writes those fields.


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