RockteMQ-AI commented on issue #4530: URL: https://github.com/apache/rocketmq-dashboard/issues/4530#issuecomment-5716641738
**Issue Evaluation** Category: `bug` | Status: **Confirmed** The reported pagination issue has been verified against the current codebase on the `rocketmq-studio` branch. **Root Cause:** Three methods in `TencentInstanceProvider` stop pagination when the current page returns null/empty data, without consulting `TotalCount`: - `listTopics()` (line 220-222) - `listConsumerGroups()` (line 442) - `listSubscriptions()` via `DescribeTopicListByGroup` When Tencent returns an empty `Data` array but `TotalCount` indicates more rows exist (e.g., `TotalCount=2` with `offset=0` returning 0 rows), the code treats this as end-of-list instead of an incomplete/contradictory response. **Impact:** Studio can silently return partial Topic, Consumer Group, and subscription inventories as complete. During administration or incident diagnosis, missing resources appear absent rather than unavailable. **Severity:** Medium — data completeness issue that can mislead operators during troubleshooting. **Code Location:** - `server/src/main/java/org/apache/rocketmq/studio/provider/tencent/TencentInstanceProvider.java` - Lines 220-222, 442, and the `DescribeTopicListByGroup` pagination loop **Suggested Fix:** When `TotalCount` is known and non-negative, treat an empty/short page with `offset + returned < TotalCount` as an error (502-style) rather than end-of-list. Preserve the existing fallback when `TotalCount` is missing. An automated fix proposal can be generated. Reply `/approve` to proceed with PR generation. --- *Automated evaluation by github-manager-bot* -- 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]
