Loyal-Young opened a new pull request, #6910: URL: https://github.com/apache/rocketmq-dashboard/pull/6910
## What is the purpose of the change Fixes #6000. Consumer-group CSV export wrote the backend's `-1` unknown-lag sentinel as a spreadsheet cell, although the page and server exporter describe that value as unavailable. The same CSV already writes `unknown` for an unavailable online-instance count. The earlier PR #6001 was automatically closed after inactivity without merging. This patch was reproduced on the current `rocketmq-studio` branch and verified independently. ## Brief changelog - Use the shared `isLagAvailable` predicate to emit `unknown` for unavailable Total Lag. - Preserve measured zero and positive lag as numeric CSV cells, without adding locale separators. - Add regression cases for unknown, zero, and measured lag. This does not alter the separate Delay Seconds column; #6514 already addresses that field. ## Verification - New focused test failed before the change (CSV contained the escaped `-1` sentinel). - `vitest run src/services/consumerService.test.ts`: 27 passed after the change. - TypeScript `tsc --noEmit`: passed. - ESLint on the two changed files: passed without warnings. - `git diff --check`: passed. ## AI assistance AI assistance helped trace the CSV column to the shared lag sentinel contract and draft the regression cases. I reviewed the current branch behavior and ran the checks above. -- 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]
