tju-yxq opened a new issue, #3150:
URL: https://github.com/apache/rocketmq-dashboard/issues/3150

   ## Problem
   
   The Consumer Group page has a Push/Pop filter, but the backend paged 
endpoint does not support it.
   
   Current behavior:
   
   - `/api/groups/page` accepts only `instanceId`, `clusterId`, `search`, 
`page`, and `pageSize`.
   - `web/src/pages/instance/consumer.tsx` fetches one page and then filters 
that page in memory with `visibleConsumerGroups(groups, modeFilter)`.
   - The repository query in `RocketMQMetadataProvider.listConsumerGroupsPage` 
also has no subscription-mode condition.
   
   This makes the filter incorrect for real inventories:
   
   - switching to `Push` or `Pop` filters only the rows on the current page, 
not the whole inventory;
   - the returned `total` remains the unfiltered total;
   - a page can render zero rows while the pagination still says there are many 
results;
   - later pages are unreachable for the selected mode because they may contain 
no matching rows;
   - CSV export uses the selected mode, so exported data and the visible page 
no longer describe the same query.
   
   ## Expected behavior
   
   - `/api/groups/page` should accept an optional `subscriptionMode` filter.
   - Apache-backed groups should filter in the database query, preserving 
correct totals and paging.
   - Cloud providers that return provider-native pages should apply the same 
filter without breaking existing search/pagination behavior.
   - The frontend should send the selected mode to the backend and stop 
filtering only the current page locally.
   - CSV export should continue to use the same backend-visible mode filter.


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