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

   **Issue Evaluation**
   
   Category: `bug` | Status: **Confirmed**
   
   The reported issue has been verified against the described code paths.
   
   **Root Cause:** `web/src/pages/instance/message.tsx` recomputes 
`currentQueryParams` from live form inputs on every render (lines 429-434). The 
pagination handler at lines 1206-1214 passes these live params directly to 
`executeQuery` without snapshotting the committed query at search time. No 
input handler clears results or snapshots the query except mode change and 
reset.
   
   **Impact:** The message list — one of the most-used pages — presents a mixed 
result set when an operator edits form inputs between pagination clicks. Page N 
may show results from a different query than Page 1, with a misleading total 
count.
   
   **Severity:** Medium — UX/data consistency issue on a high-traffic page. No 
data loss or corruption, but operators may draw incorrect conclusions from 
mixed results.
   
   **Linked PR:** #4005 — The fix approach (snapshotting committed query params 
and using them for pagination) is correct.
   
   ---
   *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