tju-yxq opened a new pull request, #2602:
URL: https://github.com/apache/rocketmq-dashboard/pull/2602

   Closes #2601
   
   ## Summary
   
   - keep message-query history recording for direct queries
   - record history only for the first page of a paginated message query
   - keep provider query and page slicing behavior unchanged
   - add a focused test that first and later pages both query the provider, 
while only page 1 writes history
   
   ## Why
   
   Every paginated request previously reused the direct query path, which 
records a history row after each provider query. Paging through one result 
therefore created duplicate `rmq_message_query` rows and inflated server 
history/summary counts.
   
   ## Tests
   
   - focused: `MessageServiceTest, MessageControllerTest` — 15 tests passed, 
Checkstyle 0 violations
   - backend full suite: 1,627 tests run; only the 2 known pre-existing 
`RocketMQMessageProviderTest` failures remain
   - `git diff --check`: passed
   
   Production changes: 11 additions / 2 deletions. The full PR is 32 additions 
/ 2 deletions. This is a real correctness/data-quality fix, but production 
changed lines are below the separate 100-line target, so the tracking ledger 
records it as a candidate rather than a complete group.


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