RockteMQ-AI commented on issue #2285: URL: https://github.com/apache/rocketmq-dashboard/issues/2285#issuecomment-5311675009
**Issue Evaluation** Category: `type/enhancement` | Status: **Evaluated** **Feasibility:** Feasible **Scope:** Managed Topic metadata API, UI topic list, and related tests **Compatibility:** No breaking changes — additive pagination support The current implementation loads all matching local records, filters system topics in memory, and returns an unbounded list. This is correct but inefficient for deployments with many topics. Adding stable `page`/`pageSize` support with the system-topic exclusion applied before page slicing is a sound improvement. **Suggested approach:** - Push the system-topic exclusion into the SQL query (e.g., exclude system topics in WHERE clause) - Add `page`/`pageSize` parameters with sensible defaults - Update the UI topic list component to use paginated API - Add regression tests for pagination boundaries and system-topic filtering --- *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]
