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

   **Issue Evaluation**
   
   Category: `bug` | Status: **Confirmed**
   
   The reported issue has been verified against the current codebase.
   
   **Root Cause:** `QueryHistoryService.getMessageQueryResults` catches 
`JsonProcessingException` when deserializing the result snapshot and returns an 
empty list (`List.of()`), masking a persistence-level failure as a legitimate 
empty result.
   
   **Impact:** When a stored snapshot becomes unreadable (e.g., after schema 
changes or serialization errors), the API returns `200` with an empty list 
instead of surfacing the error. Users see "0 messages" with no indication that 
data was lost or corrupted.
   
   **Severity:** Medium - Data integrity issue that silently masks failures
   
   **Verification:** Code inspection confirms the problematic pattern at lines 
152-156 of `QueryHistoryService.java`. The fix in PR #4791 correctly addresses 
this by changing the error handling behavior.
   
   **Recommendation:** This is a valid bug with a proposed fix. Review PR #4791 
for merge.
   
   ---
   *Automated evaluation by RockteMQ-AI*


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