RockteMQ-AI commented on issue #2240: URL: https://github.com/apache/rocketmq-dashboard/issues/2240#issuecomment-5302820673
**Issue Evaluation** Category: `bug` | Status: **Evaluated — PR already in progress** The overlapping-poll concern is valid. However, the `useVisiblePolling` hook does not yet exist in `master` or the `rocketmq-studio` branch. **Current state:** - The hook is not present in any existing source tree (checked `master`, `rocketmq-studio`, and `develop` branches). - PR #2241 (`codex/wang-visible-poll-serial`) introduces `web/src/hooks/useVisiblePolling.ts` together with comprehensive tests covering the exact scenarios described here: overlapping interval/visibility triggers, rejection recovery, hidden-page behavior, and unmount cleanup. **Assessment:** - The issue correctly identifies a real class of bugs (overlapping async polls with out-of-order state writes). - PR #2241 addresses the concern proactively by introducing the hook with an in-flight guard (`inFlight` ref) that prevents concurrent polls. - No additional action needed on this issue once PR #2241 is reviewed and merged. **Recommendation:** This issue can be closed as completed when PR #2241 is merged. --- *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]
