yuluo-yx opened a new pull request, #4395: URL: https://github.com/apache/rocketmq-dashboard/pull/4395
<!-- Make sure the base branch is `master`: that is the RocketMQ Studio trunk. --> ### Which Issue(s) This PR Fixes - Fixes #3134 - Replaces #3135, closed by the branch switch in #4379. ### Brief Description This is a resubmission of #3135 against `master`, following the branch switch notice in #4379. The fork branch is unchanged; `master` is now the RocketMQ Studio trunk. ## Summary - add a read-only Consumer Group configuration comparison drawer for two RocketMQ instances - load complete paginated inventories and classify groups as matching, drifted, source-only, or target-only - compare only stable group settings while excluding lag, delay, clients, runtime subscriptions, timestamps, and instance identity - support source/target swapping, field-level details, status/search filters, and filtered CSV export - add Chinese and English UI text plus focused utility and component coverage ## Design boundaries This feature deliberately does not copy or mutate Consumer Groups. Runtime observations are excluded because they naturally differ between instances and would create false drift. Requests use a monotonically increasing identifier so an older response cannot replace a newer comparison. ### How Did You Test This Change? ## 原提交验证记录 - `npm test -- --run src/utils/consumerGroupConfigComparison.test.ts src/components/__tests__/ConsumerGroupConfigComparisonDrawer.test.tsx`: 2 files, 14 tests passed - `npm run lint`: 0 errors; 10 pre-existing warnings outside this change - `npm run build`: TypeScript and Vite production build passed; 8040 modules transformed - `git diff --check`: passed - common base `36126024cbf75ae2e0f27cc074b2f39ceb6c8b6e`: 6 files, 942 additions, 1 deletion Fixes #3134 ### Checklist - [x] One coherent change; unrelated modifications are not bundled in - [x] Commit subject follows Conventional Commits (`feat:` / `fix:` / `refactor:` / `chore:` / `docs:` / `perf:`) - [x] Tests added or updated for non-trivial changes, test methods named `...Test` - [x] New UI text has both Chinese and English entries under `web/src/i18n/`, or the change does not add UI text - [x] Architecture constraints stay green (`mvn test` runs the ArchUnit checks), or the original verification scope is stated above - [x] New source files carry the ASF license header, or no new source files were added - [x] Documentation touched where behaviour changed (README / `docs/` / in-app help), or no documentation change was required -- 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]
