tju-yxq opened a new pull request, #3937: URL: https://github.com/apache/rocketmq-dashboard/pull/3937
## What changed The instance page fetched the entire filtered inventory and rendered it in one unbounded table. It also had no vendor selector, even though the data model and page already distinguish Apache, Aliyun, and Tencent instances. This change adds vendor filtering and server-side pagination: - add `GET /api/instances/page` with a paged result contract; - accept the existing `type` and `search` filters plus a new `vendor` filter; - normalize and validate the vendor value; - validate page and page size bounds; - reuse the existing instance list pipeline, including count enrichment, region-name resolution, and stable vendor/region/name ordering; - add a typed frontend API and service wrapper; - add an Apache/Aliyun/Tencent vendor selector; - replace the unbounded table with server-side pagination; - reset to the first page when type, vendor, or search changes; - keep the existing mock mode behavior with the same page contract. Closes #3564 ## Validation - `mvn -q '-Dtest=InstanceServiceTest,InstanceControllerTest' test` — 94 tests passed - `mvn -q checkstyle:check` — passed - `npm test -- InstancePage.test.tsx --run` — 22 tests passed - `npm run lint -- --quiet` — 0 errors; 10 existing warnings remain elsewhere in the tree - `npm run build` — passed - `git diff --check` — passed The GitHub Actions status is not being described as green here; the checks above are local validation on this branch. -- 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]
