RockteMQ-AI commented on issue #3293: URL: https://github.com/apache/rocketmq-dashboard/issues/3293#issuecomment-5549851346
**Issue Evaluation** Category: `type/bug` | Status: **Confirmed** | Severity: **Medium** Verified against `rocketmq-studio` branch (base commit 3612602). **Root Cause:** In `runTraceQuery` (`web/src/pages/instance/message.tsx:643`), the `traceGenerationRef` is bumped (line 645) *before* the empty-input validation (line 647). When the input is empty, the function returns early (line 649) without resetting `traceLoading`. Any in-flight `loadMessageTrace` request is invalidated by the generation bump, so its `finally` guard (line 672: `traceGenerationRef.current === requestGeneration`) also skips the `setTraceLoading(false)` call. Result: the trace tab spinner persists indefinitely. **Impact:** Client-side UX only — the trace tab becomes unresponsive until the user issues another non-empty query or reopens the dialog. No data loss or server-side effect. **Assessment:** Well-documented with correct code analysis and a deterministic Vitest reproduction. Draft PR #3156 addresses this with the fix and regression test. --- *Automated evaluation by github-manager* -- 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]
