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

   ### Before Creating the Bug Report
   
   - [x] I searched open/closed issues and PRs for Tencent 
topic/group/subscription pagination, incomplete pages, and `TotalCount` 
handling.
   - [x] I reproduced all three paths on current `master` 
(`d50ffecc9d7e8f8f46da64198831bd7952e6974e`).
   
   ### Problem
   
   Three Tencent RocketMQ 5.x resource inventories stop pagination solely 
because the current page is null/empty or shorter than `PAGE_SIZE`, even when 
the same response's `TotalCount` proves more provider rows exist:
   
   - `DescribeTopicList` in `listTopics`;
   - `DescribeConsumerGroupList` in `listConsumerGroups`;
   - `DescribeTopicListByGroup` in group subscription/progress loading.
   
   Tencent's current API documentation defines `TotalCount` as the total query 
count for all three interfaces. A short/empty page with `offset + returned < 
TotalCount` is therefore incomplete/contradictory, not a confirmed end of the 
list.
   
   ### Impact
   
   Studio can silently return partial Topic, Consumer Group, subscription and 
consumer-progress inventories as complete. During administration or incident 
diagnosis, missing resources can look genuinely absent instead of unavailable 
from the provider response.
   
   ### Expected behavior
   
   - When a known non-negative `TotalCount` proves rows remain but the current 
page is short/empty, fail closed with a 502-style error.
   - Preserve normal complete pages and final pages.
   - When `TotalCount` is missing/negative, retain the existing short-page 
fallback behavior.
   - Keep existing mappings, enrichment, filters, and >10k pagination behavior 
unchanged.
   
   ### Red reproduction
   
   On unmodified `master`, add one `TotalCount=2` / one-row regression for each 
of the three paths and require a 502.
   
   `TencentInstanceProviderTest`: **47 tests, exactly 3 failures**. Each new 
test fails with `Expecting code to raise a throwable`; the other 44 tests pass.
   
   ### Relation to existing work
   
   #4513/#4514 cover Tencent message-query completeness; #4528 covers the 
Tencent cloud instance catalog. This issue is the separate runtime resource 
inventory layer.
   
   AI-assisted source audit and regression authoring; Tencent's current API 
documentation for all three interfaces was checked before filing.


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