RockteMQ-AI commented on issue #2568: URL: https://github.com/apache/rocketmq-dashboard/issues/2568#issuecomment-5406298391
**Issue Evaluation** Category: | Status: **Evaluated** **Feasibility:** Feasible **Scope:** Dashboard backend (NameServer controller/service/repository) + Cluster page frontend **Compatibility:** Backward-compatible if legacy endpoint is preserved This is a well-scoped performance improvement. The current unbounded for NameServer registries will degrade as inventory grows. The proposed pagination contract (page/pageSize/search/status filter) with SQL-level filtering and deterministic ordering is the correct approach. **Suggestions:** - Consider using or a similar pagination utility if the project already uses one, for consistency with other paginated endpoints. - The legacy full-list endpoint should be deprecated with a clear migration path rather than kept indefinitely. - Frontend should handle empty pages and invalid page numbers gracefully (redirect to page 1). This enhancement is suitable for community contribution. --- *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]
